mam/mamweb
11
0
Fork 0
Code Issues Pull requests 8 Projects Wiki Activity Actions

editorconfig (#1011) #9

Merged
zelvuska merged 4 commits from editorconfig into master 2022-11-16 08:59:12 +01:00
Conversation 10 Commits 4 Files changed 1 +21
zelvuska commented 2022-11-15 13:18:04 +01:00
Owner
Copy link
No description provided.
zelvuska added 2 commits 2022-11-15 13:18:05 +01:00
zelvuska commented 2022-11-15 13:18:56 +01:00
Author
Owner
Copy link

Napadá mě, jestli si nedefinovat i něco na html, protože některá vypadají strašně.

Napadá mě, jestli si nedefinovat i něco na html, protože některá vypadají strašně.
zelvuska reviewed 2022-11-15 13:21:01 +01:00
.editorconfig
@ -0,0 +9,4 @@
[*.py]
indent_style = tab
# I do not think we prescribe, how big a tab is.
indent_size = unset
zelvuska commented 2022-11-15 13:21:01 +01:00
Author
Owner
Copy link

A nechceme to nastavit? Protože pak člověk zarovná třeba komentáře, nebo kód k závorce, a jinde to bude rozjeté.

A nechceme to nastavit? Protože pak člověk zarovná třeba komentáře, nebo kód k závorce, a jinde to bude rozjeté.
ledoian commented 2022-11-15 16:12:59 +01:00
Owner
Copy link

A máme takový kód? Kromě římských čísel, která stejně žádným obecným pravidlem postihnout IMHO nejdou. A stejně obecně pokud potřebuju, aby to skutečně bylo zarovnané, tak musím použít stejné množství tabů na začátku a zbytek doplnit mezerami, jinak to „sémanticky“ je rozbité pořád (a nevidím úplně výhodu v tom snažit se před tím zavírat oči)…

Osobně jsem u Pythonu spíš názoru, že má existovat konečně mnoho úrovní odsazení a nesnažit se zarovnávat moc freestylově.

A máme takový kód? Kromě římských čísel, která stejně žádným obecným pravidlem postihnout IMHO nejdou. A stejně obecně pokud potřebuju, aby to skutečně bylo zarovnané, tak musím použít stejné množství tabů na začátku a zbytek doplnit mezerami, jinak to „sémanticky“ je rozbité pořád (a nevidím úplně výhodu v tom snažit se před tím zavírat oči)… Osobně jsem u Pythonu spíš názoru, že má existovat konečně mnoho úrovní odsazení a nesnažit se zarovnávat moc freestylově.
zelvuska commented 2022-11-15 16:16:05 +01:00
Author
Owner
Copy link

Á, to mi nedošlo, že by se to mělo zarovnat taby na odsazení a pak už jen mezerami. To máme asi někde špatně…

Á, to mi nedošlo, že by se to mělo zarovnat taby na odsazení a pak už jen mezerami. To máme asi někde špatně…
ledoian commented 2022-11-15 16:18:58 +01:00
Owner
Copy link

No, i tak by mě zajímalo, kde něco tak šíleného tvoříme. Na klasické věci mi zatím vždycky stačily klasické taby bez potřeby zarovnávat k závorkám; jediná věc, která mě napadá, jsou složité podmínky a tam mi (od stolu) dává smysl se spíš pokusit je zpřehlednit než udržovat…

No, i tak by mě zajímalo, kde něco tak šíleného tvoříme. Na klasické věci mi zatím vždycky stačily klasické taby bez potřeby zarovnávat k závorkám; jediná věc, která mě napadá, jsou složité podmínky a tam mi (od stolu) dává smysl se spíš pokusit je zpřehlednit než udržovat…
ledoian marked this conversation as resolved
ledoian commented 2022-11-15 16:09:01 +01:00
Owner
Copy link

Napadá mě, jestli si nedefinovat i něco na html, protože některá vypadají strašně.

A máme někde vyloženě problém s odsazováním? To je aktuálně asi to jediné, co editorconfig umí sdělit…

> Napadá mě, jestli si nedefinovat i něco na html, protože některá vypadají strašně. A máme někde vyloženě problém s odsazováním? To je aktuálně asi to jediné, co editorconfig umí sdělit…
zelvuska commented 2022-11-15 16:13:33 +01:00
Author
Owner
Copy link

No právě mi přijde, že jeden z problémů našich html je to, že jsou odsazované úplně náhodně a tak není vidět, kde která část končí. A trochu mi přijde, že tomu pomůže, když bude odsazování stejně široké (i když je to spíše problém toho, že jsou různé věci odsazené úplně náhodně vůči okolí).

No právě mi přijde, že jeden z problémů našich html je to, že jsou odsazované úplně náhodně a tak není vidět, kde která část končí. A trochu mi přijde, že tomu pomůže, když bude odsazování stejně široké (i když je to spíše problém toho, že jsou různé věci odsazené úplně náhodně vůči okolí).
ledoian commented 2022-11-15 16:15:50 +01:00
Owner
Copy link

No právě mi přijde, že jeden z problémů našich html je to, že jsou odsazované úplně náhodně a tak není vidět, kde která část končí. A trochu mi přijde, že tomu pomůže, když bude odsazování stejně široké (i když je to spíše problém toho, že jsou různé věci odsazené úplně náhodně vůči okolí).

Jakože v rámci jednoho souboru? Máš příklad?

Ano, výsledné HTML stejně bude maglajs, protože to DjangoTemplates hází do sebe bez ladu a skladu, ale to taky není problém editorů…

> No právě mi přijde, že jeden z problémů našich html je to, že jsou odsazované úplně náhodně a tak není vidět, kde která část končí. A trochu mi přijde, že tomu pomůže, když bude odsazování stejně široké (i když je to spíše problém toho, že jsou různé věci odsazené úplně náhodně vůči okolí). Jakože v rámci jednoho souboru? Máš příklad? Ano, výsledné HTML stejně bude maglajs, protože to DjangoTemplates hází do sebe bez ladu a skladu, ale to taky není problém editorů…
ledoian commented 2022-11-15 16:16:25 +01:00
Owner
Copy link

Ale jako whatever, pokud si myslíme, že tam mají být taby, tak to do toho configu napíšu :-)

Ale jako whatever, pokud si myslíme, že tam mají být taby, tak to do toho configu napíšu :-)
zelvuska commented 2022-11-15 16:20:43 +01:00
Author
Owner
Copy link

Tak třeba base.html je dost náhodné… Ale jak na to tak koukám, tak to není tak špatné, jak jsem měl pocit…

Tak třeba base.html je dost náhodné… Ale jak na to tak koukám, tak to není tak špatné, jak jsem měl pocit…
ledoian commented 2022-11-15 16:24:21 +01:00
Owner
Copy link

A co s těmi html chceme dělat? Taby šířky 2? 2 mezery? Nic širšího než 4 mi nedává smysl, protože to často je hodně zanořených bloků, i proto bych asi bral tu šířku 2, a tam už mi ty taby a mezery poněkud splývají…

A co s těmi html chceme dělat? Taby šířky 2? 2 mezery? Nic širšího než 4 mi nedává smysl, protože to často je hodně zanořených bloků, i proto bych asi bral tu šířku 2, a tam už mi ty taby a mezery poněkud splývají…
zelvuska commented 2022-11-16 01:05:56 +01:00
Author
Owner
Copy link

Dal bych 2 mezery (to je asi nejčastější, co tam aktuálně máme) (taby tam myslím nikde nejsou, oni se možná nelíbí htmlku...)

Dal bych 2 mezery (to je asi nejčastější, co tam aktuálně máme) (taby tam myslím nikde nejsou, oni se možná nelíbí htmlku...)
ledoian commented 2022-11-16 01:18:06 +01:00
Owner
Copy link

HTML je to AFAIK naprosto jedno, ostatně i v tom zmíněném base.html ty taby někde jsou… OK, budou dvě mezery.

HTML je to AFAIK naprosto jedno, ostatně i v tom zmíněném `base.html` ty taby někde jsou… OK, budou dvě mezery.
ledoian added 1 commit 2022-11-16 01:19:13 +01:00
ledoian changed title from editorconfig to editorconfig (#1011) 2022-11-16 01:22:11 +01:00
ledoian added 1 commit 2022-11-16 01:37:55 +01:00
zelvuska commented 2022-11-16 08:59:07 +01:00
Author
Owner
Copy link

Už mě nic nenapadá a funguje mi to :)

Už mě nic nenapadá a funguje mi to :)
zelvuska merged commit 577860985a into master 2022-11-16 08:59:12 +01:00
zelvuska deleted branch editorconfig 2022-11-16 08:59:25 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: mam/mamweb#9
No description provided.
Delete branch "editorconfig"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?