Skip to content

docs(linter): add eslint/camelcase rule to unsupported list in favor of another rule.#17571

Merged
graphite-app[bot] merged 1 commit intomainfrom
camel-through-eye-of-needle
Jan 2, 2026
Merged

docs(linter): add eslint/camelcase rule to unsupported list in favor of another rule.#17571
graphite-app[bot] merged 1 commit intomainfrom
camel-through-eye-of-needle

Conversation

@connorshea
Copy link
Member

Preference is to implement this via @typescript-eslint/naming-convention instead.

See this comment for further context: #16908 (comment)

Copilot AI review requested due to automatic review settings January 1, 2026 22:50
@github-actions github-actions bot added the C-docs Category - Documentation. Related to user-facing or internal documentation label Jan 1, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds the eslint/camelcase rule to the unsupported rules list, indicating that the preference is to implement naming conventions via @typescript-eslint/naming-convention instead, which provides more flexibility for enforcing camelCase naming patterns.

Key Changes

  • Added eslint/camelcase to the unsupported rules list with a rationale explaining the preference for implementing this functionality through @typescript-eslint/naming-convention

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@camc314 camc314 changed the title docs(linter): Add eslint/camelcase rule to unsupported list in favor of another rule. docs(linter): add eslint/camelcase rule to unsupported list in favor of another rule. Jan 2, 2026
@camc314 camc314 self-assigned this Jan 2, 2026
@camc314 camc314 added the 0-merge Merge with Graphite Merge Queue label Jan 2, 2026
Copy link
Contributor

camc314 commented Jan 2, 2026

Merge activity

…of another rule. (#17571)

Preference is to implement this via [`@typescript-eslint/naming-convention`](https://typescript-eslint.io/rules/naming-convention/) instead.

See this comment for further context: #16908 (comment)
@graphite-app graphite-app bot force-pushed the camel-through-eye-of-needle branch from fd8e44e to 7e25a3a Compare January 2, 2026 12:49
@graphite-app graphite-app bot merged commit 7e25a3a into main Jan 2, 2026
18 checks passed
@graphite-app graphite-app bot deleted the camel-through-eye-of-needle branch January 2, 2026 12:55
@graphite-app graphite-app bot removed the 0-merge Merge with Graphite Merge Queue label Jan 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C-docs Category - Documentation. Related to user-facing or internal documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants