Formátování, vysvětlivky#70
Conversation
|
Já vím, je to obrovský diff, ale většina z toho jsou jen změny formátování. Prosím, projděte si ty moje změny a kdyžtak napište, co by se mělo změnit. |
|
Ahoj, jsou nějaké silné důvody k změně formátování v takovémhle rozsahu (ještě jsem nekoukal na samotný obsah, až večer kdyžtak)? Jestli ano, tak i tak by to za mě měl být separátní PR bez změn obsahu a s aspoň drobnou diskuzí, ať se nemíchají různé věci dohromady, kor kdyč se squshuje. Případně shrneš, prosím, co a proč jsi přeformátoval? |
|
Aha, pardon, mně se do tohodle připletly emaily z #69 |
|
Prohnal jsem všechny dokumenty skrz Prettier, ale jo, asi by bylo lepší to udělat separátně. Takže mám vytáhnout to formátování do odděleného PR? Editor mi to udělal automaticky při uložení a nějak mi to nedošlo. |
Lastaapps
left a comment
Lastaapps
left a comment
There was a problem hiding this comment.
Super, díky za to, vypadá to o dost líp. Jsem rád, žes to jenom nevzal formáterem, ale fakt jsi na to koukal a části přepsal.
Nicméně aby se tohle nové formátování zachovalo, by bylo strašně fajn udělat alespoň jednoduchý CONTRIBUTING.md s popisem, čím a jak se má formátovat, a pak .editorconfig (nebo co je teď standard) s počtem mezer na tab a max délkou řádku (+ další eventuelně). Kdyby sis chtěl víc hrát, můžeš potenciálně přidat pipelinu, co bude formátování kontrolovat a vynucovat.
| `name="=?utf-8?B?<STRING>?="`. Daný string dekódujete na bajty pomocí base64, a ty pak dekódujete na text pomocí utf-8 | ||
| Pokud obsahuje název pouze ASCII znaky, parametr `name` vypadá takto: | ||
| `name=<filename>`. V opačném případě vypadá takto: | ||
| `name="=?utf-8?B?<STRING>?="`. Daný string dekódujete na bajty pomocí base64, a |
There was a problem hiding this comment.
Když už tu jsou i něaké drobné změny, tušíme, jaká verze Base64 se tady použije (padding/URL/...)
| ``` | ||
|
|
||
|
|
||
| <!-- TODO: tohle mi přijde redundantní. Možná odebrat? |
There was a problem hiding this comment.
Mně osobně to pro konzistenci neuráží
|
|
||
| Vyšle request o login. | ||
|
|
||
| Url na začátku je vysvětleno v [README](../README.md). |
There was a problem hiding this comment.
Kdybys chtěl být hodně pedantic, tak ještě můžeš sedem nahradit url a Url za URL
| "typ": "JWT", | ||
| "x5t": "XXXXXXXXXXXXXXXXXXXXXXXXXXX" | ||
| "typ": "oi_reft+jwt", | ||
| } |
There was a problem hiding this comment.
Asi souhlasím s odstraněním. Pokud by to tam zůstalo, tak max jako fun fact pro čtenáře, aby si třeba vyhledal digitální podpisy.
|
A chápu teda správně, že bys chtěl nejdřív mergnout tohle a pak až #68? |
|
Jo, kdyby se to rozdělilo tak nejdřív mergenout formátování jako takové a pak ty obsahové změny. |
Ten pipeline mě taky napadl, asi by to nebylo od věci. Tak tyhle věci dám pak do dalšího PR. |
2 participants
PR k #68.