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

fix(lint/noUndeclaredVariables): recognize Uppercase #109

Merged
merged 1 commit into from
Sep 1, 2023

Conversation

Conaclos
Copy link
Member

@Conaclos Conaclos commented Sep 1, 2023

Summary

Fix #106.
The list of TypeScript globals was sorted in a natural order instead of a byte order.

Test Plan

To avoid this issue I added a test that asserts the order of the list.

@Conaclos Conaclos temporarily deployed to Website deployment September 1, 2023 12:09 — with GitHub Actions Inactive
@github-actions github-actions bot added A-Linter Area: linter A-Website Area: website L-JavaScript Language: JavaScript and super languages A-Changelog Area: changelog labels Sep 1, 2023
Copy link
Member

@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.

Ah yeah, they must be sorted because we use a binary search

@ematipico
Copy link
Member

cargo codegen-website should fix the CI

@Conaclos Conaclos force-pushed the conaclos/lint/noUndeclaredVaraibles/106 branch from 1914c59 to 0d2aac1 Compare September 1, 2023 12:28
@Conaclos Conaclos temporarily deployed to Website deployment September 1, 2023 12:28 — with GitHub Actions Inactive
@Conaclos
Copy link
Member Author

Conaclos commented Sep 1, 2023

Just wondering: is there a way to tell Astro to "include" CHANGELOG.md in changelog.mdx instead of generating the file?

@Conaclos Conaclos merged commit 3e49466 into main Sep 1, 2023
@Conaclos Conaclos deleted the conaclos/lint/noUndeclaredVaraibles/106 branch September 1, 2023 12:34
@Conaclos Conaclos restored the conaclos/lint/noUndeclaredVaraibles/106 branch September 1, 2023 18:46
@Conaclos Conaclos deleted the conaclos/lint/noUndeclaredVaraibles/106 branch September 1, 2023 18:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Changelog Area: changelog A-Linter Area: linter A-Website Area: website L-JavaScript Language: JavaScript and super languages
Projects
None yet
Development

Successfully merging this pull request may close these issues.

🐛 lint/correctness/noUndeclaredVariables doesn't recognize type Uppercase
2 participants