-
Notifications
You must be signed in to change notification settings - Fork 5
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
new degrees config format #285
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Prova a formattare usando prettier, che mi sa che molti spazi sono diversi per quello
Dipende da cartabinaria/config#62 |
Comunque ho rotto qualcosa sulle dash che non sono info e info magistrale perché sono tutte grige. Non mergiate che provo a fixare |
@VaiTon risolto anche il problema che alcune dash erano tutte grige. Ora mancherebbe il check esplicito sui null che mi hai suggerito. Io non ho problemi a farlo, ma volevo capire se era realmente necessario o più una questione di stile |
Se è una questione di stile, io preferisco alla Samu. |
In generale il fatto di utilizzare i valory "truthy" in JS mi da molto fastidio (e non solo a me, c'è un check di eslint fatto apposta), ma se vogliamo tenerlo così mi va bene comunque. Sappiate però che ci sono tanti valori (non solo Footnotes |
Mi fido più di te riguardo a questo, tra stasera e domani faccio i check per i null più espliciti |
@VaiTon direi che ci sono. Mannaggia a js |
No description provided.