Skip to content

chore: Move linting rules to root level#10116

Merged
lizschwab merged 1 commit into
RaspberryPiFoundation:mainfrom
lizschwab:7760
Jul 9, 2026
Merged

chore: Move linting rules to root level#10116
lizschwab merged 1 commit into
RaspberryPiFoundation:mainfrom
lizschwab:7760

Conversation

@lizschwab

Copy link
Copy Markdown
Contributor

The basics

The details

Resolves

Fixes #7760

Proposed Changes

Move eslint rules from package-level eslint.config.mjs files to root-level

Reason for Changes

This sets us up for moving plugins from blockly-samples over to the blockly monorepo

Test Coverage

No unit tests. I tested my changes locally by intentionally adding linting errors to each of the two projects (core and docs) and made sure it raised errors on those changes and only those changes.
Screenshot 2026-07-08 at 4 17 25 PM

@lizschwab lizschwab requested a review from a team as a code owner July 8, 2026 23:34
@lizschwab lizschwab requested a review from gonfunko July 8, 2026 23:34
@github-actions github-actions Bot added the PR: feature Adds a feature label Jul 8, 2026
@lizschwab

Copy link
Copy Markdown
Contributor Author

I'm not actually sure if this should be a feat -- maybe build would fit better?

@gonfunko gonfunko left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This is great, thanks! I'd probably use chore for the PR title, but up to you.

@lizschwab lizschwab changed the title feat: Move linting rules to root level chore: Move linting rules to root level Jul 9, 2026
@github-actions github-actions Bot added PR: chore General chores (dependencies, typos, etc) and removed PR: feature Adds a feature labels Jul 9, 2026
@lizschwab lizschwab merged commit 8a4d29b into RaspberryPiFoundation:main Jul 9, 2026
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR: chore General chores (dependencies, typos, etc)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Set up linting

2 participants