Siilwyn / css-declaration-sorter-atom

Atom package to sort CSS declarations.

Home Page:https://atom.io/packages/css-declaration-sorter

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

[discussion] enable custom config for each project

ClemensSahs opened this issue · comments

Here I write some ideas for the feature custom config file, this is a next evolution for #109

naming

paths

  • ~/
  • currentProjectDir/

I think this is a duplicate of #49?

I think to start with one file in the current project directory makes the most sense, don't want to read files outside of your project. Perhaps a css-declaration-sorter-atom.json file?

Personally not a huge fan of this feature either way since it makes more sense to do this in a dev/build npm script using the postcss cli and css-declaration-sorter directly.

I think this is a duplicate of #49?

Is fixed with #109 in my mind

I think to start with one file in the current project directory makes the most sense, don't want to read files outside of your project.

Yes thats why I want a discussion over the possible points ;)

Perhaps a css-declaration-sorter-atom.json file?

in My mind package.json is the best point. But I think we need a second/fallback file for the case that a user don't use npm/yarn. And css-declaration-sorter-atom.json is more focus on our/you project. I'm a fan of much generic naming as possible. (Like PSR and framework cooperate)
We use postcss and are a sorter. If postcss-sorting.json to much similar to atom-postcss-sorting.
I will propose postcss-sorter.json as fallback file.

Personally not a huge fan of this feature either way since it makes more sense to do this in a dev/build npm script using the postcss cli and css-declaration-sorter directly.

Yes I now, but this is not in any case possible. I prefer githook pre-commit for this. But in some project we are not able to refactor everything. ;)

What do you think of css-declaration-order.json?
It's very descriptive of the content and as far as I can see doesn't conflict with other extensions.