Skip to content
This repository was archived by the owner on Feb 9, 2023. It is now read-only.

Fix babel config issues #258

Merged
merged 3 commits into from
May 2, 2022
Merged

Fix babel config issues #258

merged 3 commits into from
May 2, 2022

Conversation

hudochenkov
Copy link
Member

Which issue, if any, is this issue related to?

Fixes #247 and #250

Is there anything in the PR that needs further explanation?

Issues was in incorrect config object passed to Babel, if there is existing user Babel config with plugins/presets. As @astone123 pointed out.

I've updated Babel to the latest version just in case. It caused adding index in JSONs.

@hudochenkov
Copy link
Member Author

@ybiquitous @jeddy3 folks, maybe you have time to check this PR? It would really helped me out, as stylelint doesn't work in my project ¯\_(ツ)_/¯

@ybiquitous
Copy link
Member

@hudochenkov OK. Honestly, I'm not so familiar with this project, but anyway I try to review it. 👀

@hudochenkov hudochenkov requested a review from ybiquitous May 2, 2022 11:39
@hudochenkov
Copy link
Member Author

Honestly, I'm not so familiar with this project

@ybiquitous Me neither :)

Thank you for your review!

Copy link
Member

@ybiquitous ybiquitous left a comment

Choose a reason for hiding this comment

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

@hudochenkov Looks good, thank you 👍🏼

@ybiquitous
Copy link
Member

Is the next version v0.37.3? The current is v0.37.2.

@ybiquitous
Copy link
Member

@hudochenkov Do you want to release the next version?

@hudochenkov
Copy link
Member Author

@ybiquitous yes, I'll release it later today. Thank you!

@hudochenkov hudochenkov merged commit 299a9b6 into master May 2, 2022
@hudochenkov hudochenkov deleted the fix-babel-config-issues branch May 2, 2022 15:59
@hudochenkov
Copy link
Member Author

  • Fixed: silent fail if Babel config is present (#258).

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

Successfully merging this pull request may close these issues.

Suppressed Babel plugin parsing error results in no linting
2 participants