Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 UTC on 18 March 2024, to get $100 off your ticket.
Problem/Motivation
Since #2865971: Use stylelint as opposed to csslint in core, we are using Stylelint instead of CSSLint. However, CSSLint is still being run on all core CI builds regardless of it not being useful at this point. This both, wastes resources and creates potential for confusion since it is quite reasonable to assume that the CI output is valid.
Proposed resolution
As we are waiting on a decision whether we can remove the .csslintrc from core on #2876298: Contrib, csslint and stylelint (discuss whether to remove .csslint from core), remove CSSLint from being run as part of the Drupal core CI builds.
Remaining tasks
- Confirm from the test output that DrupalCI doesn't execute commands related to CSSLint
Comment | File | Size | Author |
---|---|---|---|
#2 | 3166360-2.patch | 389 bytes | lauriii |
Comments
Comment #2
lauriiiComment #3
lauriiiThere's also #2866840: Use stylelint as opposed to csslint in DrupalCI for enabling Stylelint tests in DrupalCI.
Comment #4
lauriiiComment #5
tedbowOk this looks good because we are moving to Stylelint
Comment #6
alexpottCommitted and pushed 0e1565c473 to 9.1.x and 019df6c3f2 to 9.0.x and 4eff6f7457 to 8.9.x. Thanks!
As a CI change of very low risk backported to 8.9.x.