Skip to content

Todo/eslint config vue ignore camel case aria label#280

Merged
hejtful merged 3 commits intomasterfrom
todo/eslint-config-vue-ignore-camel-case-aria-label
Mar 26, 2026
Merged

Todo/eslint config vue ignore camel case aria label#280
hejtful merged 3 commits intomasterfrom
todo/eslint-config-vue-ignore-camel-case-aria-label

Conversation

@hejtful
Copy link
Copy Markdown
Contributor

@hejtful hejtful commented Mar 26, 2026

No description provided.

@hejtful hejtful requested a review from a team as a code owner March 26, 2026 09:52
@hejtful hejtful requested review from Copilot and knime-ghub-bot and removed request for a team March 26, 2026 09:52
@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Mar 26, 2026

🦋 Changeset detected

Latest commit: 085092c

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

This PR includes changesets to release 1 package
Name Type
@knime/eslint-config 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

Copy link
Copy Markdown

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

Updates the @knime/eslint-config Vue 3 ESLint preset to stop flagging the ariaLabel prop for Vue components, reducing the need for per-template eslint-disable vue/attribute-hyphenation workarounds across the monorepo.

Changes:

  • Configure vue/attribute-hyphenation to ignore the ariaLabel attribute name in Vue SFC templates.
  • Add a patch changeset for @knime/eslint-config documenting the lint behavior change.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
packages/lint/vue3.js Adds vue/attribute-hyphenation configuration to ignore ariaLabel while keeping hyphenation enforced for other attributes.
.changeset/upset-cycles-marry.md Declares a patch release for @knime/eslint-config to publish the updated Vue lint rule behavior.

@hejtful hejtful merged commit 835625e into master Mar 26, 2026
7 checks passed
@hejtful hejtful deleted the todo/eslint-config-vue-ignore-camel-case-aria-label branch March 26, 2026 10:07
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.

4 participants