Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Rename custom ESLint rule to prefer-use-i18n-polyfill #6120

Merged
merged 3 commits into from
Nov 10, 2024

Conversation

absidue
Copy link
Member

@absidue absidue commented Nov 8, 2024

Rename custom ESLint rule to prefer-use-i18n-polyfill

Pull Request Type

  • Bugfix
  • Development tooling

Description

I picked a bad name for the ESLint rule and now it appears in the auto-complete suggestions every time you type useI18n, if you have a typo VSCode even prioritises it over the polyfill. This pull request fixes that by renaming the rule to prefer-use-i18n-polyfill.

Screenshots

import-suggestions

Testing

If the linting workflow in this PR suceeds then it works, you can also run the linting yourself locally if you don't trust GitHub actions: yarn eslint-lint.

Desktop

  • OS:
  • OS Version:
  • FreeTube version:

@github-actions github-actions bot added the PR: waiting for review For PRs that are complete, tested, and ready for review label Nov 8, 2024
@FreeTubeBot FreeTubeBot enabled auto-merge (squash) November 8, 2024 17:20
absidue and others added 2 commits November 8, 2024 18:27
Co-authored-by: ChunkyProgrammer <78101139+ChunkyProgrammer@users.noreply.github.com>
@FreeTubeBot FreeTubeBot merged commit 9ab38a4 into FreeTubeApp:development Nov 10, 2024
5 checks passed
@github-actions github-actions bot removed the PR: waiting for review For PRs that are complete, tested, and ready for review label Nov 10, 2024
@absidue absidue deleted the rename-eslint-rule branch November 10, 2024 06:37
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.

5 participants