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

@babel/plugin-proposal-class-properties loose should be false #1239

Closed
wants to merge 1 commit into from

Conversation

mrgum
Copy link
Contributor

@mrgum mrgum commented Nov 7, 2023

Issue #1237

Description of changes:
This change removes the loose="true" setting on @babel/plugin-proposal-class-properties which was not being applied anyway and caused thousands of warning lines in the logs

Checklist:

  • Have you successfully deployed to an AWS account with your changes?
  • Have you successfully tested with your changes locally?
  • Is this change also required on the AWS Solution version?

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@mrgum mrgum changed the title loose should be false @babel/plugin-proposal-class-properties loose should be false Nov 7, 2023
@mrgum
Copy link
Contributor Author

mrgum commented Nov 8, 2023

this makes some tickboxes untickable so I withdraw it

@mrgum mrgum closed this Nov 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant