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: revert back to cosmiconfig from lilconfig #1035

Merged
merged 2 commits into from Oct 26, 2021
Merged

Conversation

iiroj
Copy link
Member

@iiroj iiroj commented Oct 26, 2021

This PR reverts #981 and #1033, the latter of which was a hotfix to the first. This change was only made to decrease the size of lint-staged's node modules, but instead caused lots of errors. I think it's best to go back, because I don't have the capacity to figure out what's wrong with lilconfig, and cosmiconfig worked fine.

This hopefully fixes #1034, and relates to #1032.

@iiroj
Copy link
Member Author

iiroj commented Oct 26, 2021

Ping @okonet, what do you think?

@codecov
Copy link

codecov bot commented Oct 26, 2021

Codecov Report

Merging #1035 (a90bb89) into master (612d806) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##            master     #1035   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           20        20           
  Lines          656       654    -2     
  Branches       149       149           
=========================================
- Hits           656       654    -2     
Impacted Files Coverage Δ
lib/index.js 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 612d806...a90bb89. Read the comment docs.

@iiroj iiroj merged commit e035b80 into master Oct 26, 2021
@iiroj iiroj deleted the revert-lilconfig branch October 26, 2021 18:46
@github-actions
Copy link
Contributor

🎉 This PR is included in version 11.2.6 🎉

The release is available on:

Your semantic-release bot 📦🚀

@github-actions
Copy link
Contributor

🎉 This PR is included in version 11.3.0-beta.2 🎉

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
Development

Successfully merging this pull request may close these issues.

11.2.5 breaks config loading
2 participants