Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DRAFT: Feat/sass linters #230

Merged
merged 13 commits into from
Oct 13, 2023
Merged

DRAFT: Feat/sass linters #230

merged 13 commits into from
Oct 13, 2023

Conversation

manoncarbonnel
Copy link
Contributor

@manoncarbonnel manoncarbonnel commented Oct 4, 2023

🔧 Added Stylelint scss linter with minimal configuration:

  • Concentric CSS order Stylelint plugin
  • BEM syntax for class names
  • Rules for : hue degrees in number format to preserve current format, color names, parenthesis in functions.
  • A HTML report formatter for developer experience and clarity
stylelint html report extract

ℹ️ This minimalist configuration aims to be enriched later according to the team's code conventions.

@manoncarbonnel manoncarbonnel self-assigned this Oct 4, 2023
@manoncarbonnel manoncarbonnel requested a review from a team as a code owner October 4, 2023 13:30
package.json Outdated Show resolved Hide resolved
package.json Outdated Show resolved Hide resolved
@dpellier

This comment was marked as resolved.

package.json Outdated Show resolved Hide resolved
package.json Outdated Show resolved Hide resolved
@dpellier dpellier changed the title DRAFT: Feat/sass linters Feat/sass linters Oct 11, 2023
@manoncarbonnel manoncarbonnel changed the title Feat/sass linters DRAFT: Feat/sass linters Oct 11, 2023
@ovh ovh deleted a comment from ovh-cds Oct 13, 2023
@ovh ovh deleted a comment from ovh-cds Oct 13, 2023
dpellier

This comment was marked as resolved.

@dpellier dpellier merged commit 9974e6f into master Oct 13, 2023
@dpellier dpellier deleted the feat/sass-linters branch October 13, 2023 12:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants