Skip to content

build: expand lintspaces globs#2529

Open
nschonni wants to merge 8 commits intowet-boew:masterfrom
nschonni:lint-spaces
Open

build: expand lintspaces globs#2529
nschonni wants to merge 8 commits intowet-boew:masterfrom
nschonni:lint-spaces

Conversation

@nschonni
Copy link
Copy Markdown
Member

What does this pull request(PR) do?

Expand the lintspaces globs and fix the new issues. Temporarily ignored MD and JS files, since they'll be handled by existing PRs, then can be opt'd in again

Contribution category (please only choose one)

  • Stateless or basic component
  • Interactive component
  • Helper component
  • Site wide component
  • Baseline layout
  • Page layout

General checklist

Related issues

@Garneauma
Copy link
Copy Markdown
Collaborator

Pre-approved upon successful review.

nschonni

This comment was marked as outdated.

Copy link
Copy Markdown
Collaborator

@Garneauma Garneauma left a comment

Choose a reason for hiding this comment

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

Please exclude files under the "_data/" directory. They are autogenerated, so they will always fail if they failed during your testing.

@nschonni
Copy link
Copy Markdown
Member Author

nschonni commented Jun 5, 2025

I'll see if there is an override I can make to https://github.com/wet-boew/GCWeb/blob/master/.editorconfig so it passes without the changes, but doesn't require ignoring

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