Skip to content

allowLeadingUnderscore and allowTrailingUnderscore should not conflict with ignorePattern in naming-convention rule #2785

New issue

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

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

Already on GitHub? # to your account

Merged
merged 2 commits into from
Nov 29, 2024

Conversation

dimaMachina
Copy link
Contributor

fixes #1719

Copy link

changeset-bot bot commented Nov 29, 2024

🦋 Changeset detected

Latest commit: 0cc689c

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@graphql-eslint/eslint-plugin Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@theguild-bot
Copy link
Collaborator

🚀 Snapshot Release (alpha)

The latest changes of this PR are available as alpha on npm (based on the declared changesets):

Package Version Info
@graphql-eslint/eslint-plugin 4.1.1-alpha-20241129095718-0cc689c0eb8b65d162f4c6d87030879fe63a105b npm ↗︎ unpkg ↗︎

Copy link
Contributor

💻 Website Preview

The latest changes are available as preview in: https://8e372844.graphql-eslint.pages.dev

@dimaMachina dimaMachina merged commit 99b0703 into master Nov 29, 2024
8 checks passed
@dimaMachina dimaMachina deleted the v4-17 branch November 29, 2024 10:20
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

naming-convention rule FieldDefinition.ignorePattern config still checking underscores _
2 participants