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

Add babel plugin to workaround bug in @babel/preset-env #45664

Merged
merged 1 commit into from
Feb 21, 2022
Merged

Conversation

ashharrison90
Copy link
Contributor

What this PR does / why we need it:

Which issue(s) this PR fixes:

Fixes #45624

Special notes for your reviewer:

one thing that confuses me a little is that you can apparently add comments to the babel.config.json? i guess it's not real json then, or it gets stripped out later 😕

@ashharrison90 ashharrison90 added type/bug type/chore backport v8.4.x Mark PR for automatic backport to v8.4.x labels Feb 21, 2022
@ashharrison90 ashharrison90 added this to the 8.4.2 milestone Feb 21, 2022
@ashharrison90 ashharrison90 requested a review from a team as a code owner February 21, 2022 12:02
@ashharrison90 ashharrison90 self-assigned this Feb 21, 2022
@ashharrison90 ashharrison90 requested review from zoltanbedi and removed request for a team February 21, 2022 12:02
Copy link
Contributor

@Clarity-89 Clarity-89 left a comment

Choose a reason for hiding this comment

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

Great work! 🎉 babel.config.json is a normal json file and comments shouldn't be used there. I guess this is not actually enforced when reading this file but wonder if we might get errors later at some point.

@ashharrison90 ashharrison90 merged commit 05f68c3 into main Feb 21, 2022
@ashharrison90 ashharrison90 deleted the ash/45624 branch February 21, 2022 12:50
@grafanabot
Copy link
Contributor

The backport to v8.4.x failed:

The process '/usr/bin/git' failed with exit code 1

To backport manually, run these commands in your terminal:

# Fetch latest updates from GitHub
git fetch
# Create a new branch
git switch --create backport-45664-to-v8.4.x origin/v8.4.x
# Cherry-pick the merged commit of this pull request and resolve the conflicts
git cherry-pick -x 05f68c391c67790cbf8c81c4ac295b12c808d310
# Push it to GitHub
git push --set-upstream origin backport-45664-to-v8.4.x
git switch main
# Remove the local backport branch
git branch -D backport-45664-to-v8.4.x

Then, create a pull request where the base branch is v8.4.x and the compare/head branch is backport-45664-to-v8.4.x.

ashharrison90 added a commit that referenced this pull request Feb 21, 2022
ashharrison90 added a commit that referenced this pull request Feb 21, 2022
)

* Add babel plugin to workaround bug in @babel/preset-env (#45664)

(cherry picked from commit 05f68c3)

* Change ordering
@nicolo-ribaudo
Copy link

Great work! babel.config.json is a normal json file and comments shouldn't be used there. I guess this is not actually enforced when reading this file but wonder if we might get errors later at some point.

It's a JSON5 file (as documented at https://babeljs.io/docs/en/config-files#supported-file-extensions), so it's safe to use comments there.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/frontend backport v8.4.x Mark PR for automatic backport to v8.4.x type/bug type/chore
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ReferenceError: Can't find variable: u when trying to edit a panel in Safari
4 participants