Opened 4 years ago

Closed 3 years ago

Last modified 3 years ago

#4155 closed change (rejected)

Create global csscomb config

Reported by: juliandoucette Assignee: juliandoucette
Priority: Unknown Milestone:
Module: Sitescripts Keywords:
Cc: fhd, saroyanm, greiner, trev, erikvold Blocked By:
Blocking: Platform: Unknown / Cross platform
Ready: yes Confidential: no
Tester: Unknown Verified working: no
Review URL(s):

https://codereview.adblockplus.org/29346536/

Description

Background

SScomb is a coding style formatter for CSS. You can easily write your own configuration to make your style sheets beautiful and consistent.

The main feature is sorting properties in a specific order. It was inspired by @miripiruni's PHP-based tool of the same name. This is the new JavaScript version, based on the powerful CSS parser Gonzales PE.

We can use this tool with our favorite editors (via plugin) or integrate it into our build procedures to auto-format our CSS and avoid style guide issues in review.

What to change

Add global .csscomb.json to codingtools repository

Change History (2)

comment:1 Changed 3 years ago by juliandoucette

  • Resolution set to rejected
  • Status changed from new to closed

comment:2 Changed 3 years ago by juliandoucette

I closed this ticket/review because:

  • stylelint covers more relevant rules
  • stylelint handles sass/scss better (which was not a requirement when this

ticket was created)

  • csscomb seems less configurable as a linter (which is useful for CI)
  • stylefmt covers formatting for stylelint
  • stylelint is more actively maintained
  • stylelint seems to have better editor plugins
Note: See TracTickets for help on using tickets.