Formatting for scss - #2758
Conversation
| $print: index(map-keys($breakpoints), $lower) <= | ||
| $print-index and | ||
| $print-index <= | ||
| index(map-keys($breakpoints), $upper); |
There was a problem hiding this comment.
This is weird -- why can't we format it as
index(map-keys($breakpoints), $lower) <= $print-index
and $print-index <= index(map-keys($breakpoints), $upper);
? This should fit within 120 columns with ease
There was a problem hiding this comment.
why do we even have this? can't we just use the same file as for typescript? and if not, we should just use a symlink
There was a problem hiding this comment.
without this file prettier does not work properly for scss. symlink could work, but by now the scss config does not have the arrowPens option anymore, which we want in the ts config. See #2758 (comment).
There was a problem hiding this comment.
symlinking one file is also fine for me (I just don't want a full separate copy of the file that includes attributes that don't make sense)
There was a problem hiding this comment.
I would expect that we can just have some global config for prettier. And a simple Google search reveals: yes, we can. according to https://prettier.io/docs/configuration, we can just specify everything in package.json and this config applies to all target files
3 participants
Adds formatting for
.scssusing prettier.