Skip to content
This repository has been archived by the owner on Aug 31, 2023. It is now read-only.

fix(rome_js_analyze): lint/correctness/noUndeclaredVariables incorrectly identifies AggregateError as an undeclared variable. #4365 #4366

Merged
merged 2 commits into from
Apr 11, 2023

Conversation

denbezrukov
Copy link
Contributor

@denbezrukov denbezrukov commented Apr 11, 2023

Summary

Fix #4365

Test Plan

cargo test

Changelog

  • The PR requires a changelog line

Documentation

  • The PR requires documentation
  • I will create a new PR to update the documentation

…tly identifies AggregateError as an undeclared variable. rome#4365
@netlify
Copy link

netlify bot commented Apr 11, 2023

Deploy Preview for docs-rometools canceled.

Built without sensitive environment variables

Name Link
🔨 Latest commit 689446b
🔍 Latest deploy log https://app.netlify.com/sites/docs-rometools/deploys/64350fb6074ac0000897881b

@denbezrukov denbezrukov changed the title fix(rome_js_analyze): lint/correctness/noUndeclaredVariables incorrec… fix(rome_js_analyze): lint/correctness/noUndeclaredVariables incorrectly identifies AggregateError as an undeclared variable. #4365 Apr 11, 2023
@github-actions github-actions bot added the A-Linter Area: linter label Apr 11, 2023
Copy link
Contributor

@ematipico ematipico left a comment

Choose a reason for hiding this comment

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

We have to add a line to the changelog

…tly identifies AggregateError as an undeclared variable. rome#4365
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A-Linter Area: linter
Projects
None yet
Development

Successfully merging this pull request may close these issues.

🐛 lint/correctness/noUndeclaredVariables incorrectly identifies AggregateError as an undeclared variable.
2 participants