Skip to content

Linting setup updates#541

Merged
kategengler merged 2 commits intoember-learn:masterfrom
Windvis:linting-setup-updates
Mar 9, 2026
Merged

Linting setup updates#541
kategengler merged 2 commits intoember-learn:masterfrom
Windvis:linting-setup-updates

Conversation

@Windvis
Copy link
Contributor

@Windvis Windvis commented Feb 22, 2026

To prepare the repo for .gjs files we update the linting setup first. This copied over the setup from a 6.8 blueprint since .gjs files is supported out of the box there.

  • eslint v9 + flat file config
  • prettier as a standalone tool, instead of integrated in eslint and stylelint

There are a quite of bit of changed files because I ran Prettier. Some files disabled the prettier eslint rule and weren't formatted, and .hbs files weren't being formatted either. I could try to work around that if needed, but it didn't seem worth the effort and it's contained in a single commit as well.

@netlify
Copy link

netlify bot commented Feb 22, 2026

Deploy Preview for ember-styleguide ready!

Name Link
🔨 Latest commit 95f7b9e
🔍 Latest deploy log https://app.netlify.com/projects/ember-styleguide/deploys/69a466871d346300082b9fca
😎 Deploy Preview https://deploy-preview-541--ember-styleguide.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Windvis added 2 commits March 1, 2026 17:16
This updates the eslint, prettier and stylelint setups to match the 6.8
blueprint as close as possible. This prepares the repo for .gjs files.
@Windvis Windvis force-pushed the linting-setup-updates branch from e148d27 to 95f7b9e Compare March 1, 2026 16:17
@kategengler kategengler merged commit afeac67 into ember-learn:master Mar 9, 2026
18 of 19 checks passed
@github-actions github-actions bot mentioned this pull request Mar 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants