Skip to content

Conversation

@stopflunky
Copy link

Description

This PR fixes an issue where the lint script was unintentionally
checking .js files in addition to .ts and .vue files.

Changes

  • Modified lint script in package.json to exclude *.js files
  • Modified lint:check script in package.json to exclude *.js files

Testing

  • Ran yarn run lint successfully - all files pass linting
  • Ran yarn run dev - site builds and runs correctly on localhost:8080

Fixes #135

…om lint script patterns as they should not be linted according to project requirements. Only TypeScript and Vue files should be checked. Fixes bitwarden#135
@stopflunky stopflunky requested a review from a team as a code owner January 7, 2026 23:01
@CLAassistant
Copy link

CLAassistant commented Jan 7, 2026

CLA assistant check
All committers have signed the CLA.

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@bitwarden-bot
Copy link

Thank you for your contribution! We've added this to our internal tracking system for review.
ID: PM-30551
Link: https://bitwarden.atlassian.net/browse/PM-30551

Details on our contribution process can be found here: https://contributing.bitwarden.com/contributing/pull-requests/community-pr-process.

@bitwarden-bot bitwarden-bot changed the title Fix: Lint only *.ts and *.vue files [PM-30551] Fix: Lint only *.ts and *.vue files Jan 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Unintentionally linting *.vue and *.js files.

3 participants