How to specify emptyLineBefore to never

[cenfun]

Thanks for the project.

Currently, the emptyLineBefore default to always. How to specify emptyLineBefore to never? possible reason:

• emptyLineBefore=always conflicts with rule 'declaration-empty-line-before' in stylelint-config-standard
• I prefer to emptyLineBefore=never because the count of css properties will less than 5 in most cases, the group emptyLineBefore is not necessary especially using SCSS.

What do you think of following solution

// .stylelintrc.js
const { propertyOrderingWithOptions } = require('stylelint-semantic-groups');

module.exports = {
  plugins: ['stylelint-order'],
  rules: {
    'order/properties-order': propertyOrderingWithOptions({ emptyLineBefore: 'never' })
  },
};

thanks

[theKashey]

Approved in principle 👍

I would just rename function into propertyOrderingFactory because selectorOrderFactory already exists or expose pre-constructed propertyOrderingCompact (is there a better word meaning "no spaces"? I have a feeling that it does) because you don't need, and cannot have many options - emptyLine is the single one to change with the rest being "oppionated". Well, there is also unspecified which are better to be kept "where found"

[alizadeh118]

As a quick fix I'm using this:

// .stylelintrc.js
const { propertyOrdering } = require('stylelint-semantic-groups');

propertyOrdering[0] = propertyOrdering[0].map(rule => {
    rule.emptyLineBefore = 'never';
    return rule;
});

module.exports = {
  plugins: ['stylelint-order'],
  rules: {
    'order/properties-order': propertyOrdering
  },
};

[cenfun]

@alizadeh118 thanks, I have fixed it with following

propertyOrdering.forEach((list) => {
    list.forEach((item) => {
        item.emptyLineBefore = 'never';
    });
});