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

Reduce dependencies size #978

Open
ai opened this issue May 17, 2021 · 5 comments · Fixed by #981
Open

Reduce dependencies size #978

ai opened this issue May 17, 2021 · 5 comments · Fixed by #981

Comments

@ai
Copy link
Contributor

ai commented May 17, 2021

Description

The current size of lint-staged and all its dependencies 21 MB, which is huge.

I can use Cult of Martians to send you PR with dependencies reduction and light replacements.

Steps to reproduce

$ mkdir test && cd test
$ yarn add lint-staged
$ du -sh node_modules 
21M	node_modules

Environment

  • OS: Fedora 34
  • Node.js: 16.1.0
  • lint-staged: 11.0.0
@okonet
Copy link
Collaborator

okonet commented May 17, 2021

Feel free! We have a comprehensive test suite so it should easy to try out.

@iiroj
Copy link
Member

iiroj commented May 18, 2021

I wonder if this also includes dev dependencies, we have plenty of those.

@kytta kytta mentioned this issue May 24, 2021
1 task
@github-actions
Copy link
Contributor

🎉 This issue has been resolved in version 11.2.4 🎉

The release is available on:

Your semantic-release bot 📦🚀

@iiroj iiroj reopened this Oct 23, 2021
@iiroj
Copy link
Member

iiroj commented Oct 23, 2021

Even though #981 and #1003 reduced the dependencies size by a lot, let's keep this open if we can still find other ways to slim down.

@github-actions
Copy link
Contributor

🎉 This issue has been resolved 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 a pull request may close this issue.

3 participants