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

Stop backing up the node_modules folder #962

Merged

Conversation

bertdeblock
Copy link
Contributor

@bertdeblock bertdeblock commented Sep 5, 2023

Closes #189.
Closes #598.
Closes #813.

@codecov
Copy link

codecov bot commented Sep 5, 2023

Codecov Report

Merging #962 (56f216c) into master (ec8d2ab) will decrease coverage by 0.06%.
The diff coverage is 100.00%.

❗ Current head 56f216c differs from pull request most recent head 998054e. Consider uploading reports for the commit 998054e to get more accurate results

@@            Coverage Diff             @@
##           master     #962      +/-   ##
==========================================
- Coverage   94.59%   94.53%   -0.06%     
==========================================
  Files          17       17              
  Lines         555      549       -6     
==========================================
- Hits          525      519       -6     
  Misses         30       30              
Files Changed Coverage Δ
lib/dependency-manager-adapters/npm.js 98.33% <100.00%> (-0.08%) ⬇️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@kategengler kategengler merged commit 017e7fb into ember-cli:master Sep 5, 2023
30 checks passed
@kategengler
Copy link
Member

Thank you!

@bertdeblock bertdeblock mentioned this pull request Sep 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants