Skip to content

Adopt @wagtail/stylelint-config-wagtail v2 for CSS linting - #753

Merged
thibaudcolas merged 5 commits into
wagtail:mainfrom
Srishti-j18:stylelint-config-changes
Jun 17, 2026
Merged

Adopt @wagtail/stylelint-config-wagtail v2 for CSS linting#753
thibaudcolas merged 5 commits into
wagtail:mainfrom
Srishti-j18:stylelint-config-changes

Conversation

@Srishti-j18

@Srishti-j18 Srishti-j18 commented Jun 10, 2026

Copy link
Copy Markdown
Member

Task of #743

Description

This updates bakerydemo to use @wagtail/stylelint-config-wagtail as the shared CSS linting baseline.

Changes in this PR:

  • replace stylelint-config-standard with @wagtail/stylelint-config-wagtail
  • renamed the file .stylelintrc to stylelint.config.mjs for consistency
  • keep a small set of bakerydemo-specific rule overrides where the shared config is currently too strict for the existing stylesheet

Tested locally

I updated the frontend tooling packages with: npm install stylelint @wagtail/stylelint-config-wagtail --save-dev

I tested the updated Stylelint configuration against the project CSS locally.
Checks run:

  - npm run lint:css
  - npx stylelint bakerydemo/static/css/main.css
  - npx stylelint --fix bakerydemo/static/css/main.css

AI usage

I used GPT for the configuration definition and testing approach, and I manually applied the changes and verified them locally.

@Srishti-j18 Srishti-j18 added the dependencies Pull requests that update a dependency file label Jun 10, 2026
@Srishti-j18 Srishti-j18 moved this to 🏗 In progress in GSoC 2026: Demo website redesign Jun 10, 2026
@Srishti-j18 Srishti-j18 moved this from 🏗 In progress to 👀 In review in GSoC 2026: Demo website redesign Jun 10, 2026
@Srishti-j18
Srishti-j18 force-pushed the stylelint-config-changes branch from e9703e1 to e01dbc1 Compare June 16, 2026 17:14
@Srishti-j18
Srishti-j18 force-pushed the stylelint-config-changes branch from e01dbc1 to 142feba Compare June 16, 2026 17:16

@thibaudcolas thibaudcolas left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking spot on!

@thibaudcolas thibaudcolas changed the title Adopt @wagtail/stylelint-config-wagtail Adopt @wagtail/stylelint-config-wagtail v2 for CSS linting Jun 17, 2026
@thibaudcolas
thibaudcolas merged commit 19d7753 into wagtail:main Jun 17, 2026
4 checks passed
@github-project-automation github-project-automation Bot moved this from 👀 In review to ✅ Done in GSoC 2026: Demo website redesign Jun 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

2 participants