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

feat: add support for es2020 syntax in shared eslint config #153

Merged
merged 1 commit into from
Jun 20, 2024

Conversation

adamstankiewicz
Copy link
Member

Inspired by this comment thread, this PR adds support for es2020 syntax. The primary rationale is that given we are to beginning to support globalThis syntax in shared JS libraries and MFEs, we should ensure other related ES2020 syntax is included as well, and implemented in such a way that is common/applicable as opposed to being added to individual ESLint configs. The Open edX platform officially supports the browsers defined in @edx/browserslist-config (source), where all have support for es2020 syntax.

@adamstankiewicz adamstankiewicz changed the title feat: add support for es2020 syntax in eslint feat: add support for es2020 syntax in shared eslint config Jun 20, 2024
Copy link

@arbrandes arbrandes left a comment

Choose a reason for hiding this comment

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

Great idea!

Copy link
Contributor

@bradenmacdonald bradenmacdonald left a comment

Choose a reason for hiding this comment

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

Thanks for this!

@adamstankiewicz adamstankiewicz merged commit 99261e2 into master Jun 20, 2024
5 checks passed
@adamstankiewicz adamstankiewicz deleted the ags/es2020-env branch June 20, 2024 18:20
@edx-semantic-release
Copy link

🎉 This PR is included in version 4.1.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants