Skip to content
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

Update some deprecated ESLint rules #13890

Merged
merged 1 commit into from
Aug 11, 2021

Conversation

Snuffleupagus
Copy link
Collaborator

Please see https://eslint.org/docs/rules/#deprecated where the following rules apply to the PDF.js project:

Please see https://eslint.org/docs/rules/#deprecated where the following rules apply to the PDF.js project:
 - [`no-buffer-constructor`](https://eslint.org/docs/rules/no-buffer-constructor), which we can replace with the `unicorn/no-new-buffer` rule; see https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/no-new-buffer.md
 - [`no-catch-shadow`](https://eslint.org/docs/rules/no-catch-shadow), which was replaced by the `no-shadow` rule (that we're already using).
@Snuffleupagus
Copy link
Collaborator Author

/botio lint

@pdfjsbot
Copy link

From: Bot.io (Windows)


Received

Command cmd_lint from @Snuffleupagus received. Current queue size: 0

Live output at: http://3.101.106.178:8877/694d14e14c5c46f/output.txt

@pdfjsbot
Copy link

From: Bot.io (Linux m4)


Received

Command cmd_lint from @Snuffleupagus received. Current queue size: 0

Live output at: http://54.67.70.0:8877/24c86bfebe19998/output.txt

@pdfjsbot
Copy link

From: Bot.io (Linux m4)


Success

Full output at http://54.67.70.0:8877/24c86bfebe19998/output.txt

Total script time: 2.49 mins

  • Lint: Passed

@pdfjsbot
Copy link

From: Bot.io (Windows)


Success

Full output at http://3.101.106.178:8877/694d14e14c5c46f/output.txt

Total script time: 4.44 mins

  • Lint: Passed

@timvandermeij timvandermeij merged commit f154bbf into mozilla:master Aug 11, 2021
@timvandermeij
Copy link
Contributor

Thank you for updating this!

@Snuffleupagus Snuffleupagus deleted the eslint-deprecated-rules branch August 11, 2021 21:41
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants