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

.gitignore the package.lock.json files in the packages #6631

Merged
merged 5 commits into from
May 24, 2020

Conversation

ShukantPal
Copy link
Member

This is useful because @bigtimebuddy usually requests me to remove lockfiles when they make it into my PRs.

This is useful because Matt usually requests me to remove lockfiles when they make it into my PRs.
@bigtimebuddy
Copy link
Member

That's a good call.

@codecov-commenter
Copy link

codecov-commenter commented May 19, 2020

Codecov Report

Merging #6631 into dev will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##              dev    #6631   +/-   ##
=======================================
  Coverage   82.54%   82.54%           
=======================================
  Files          36       36           
  Lines        1730     1730           
=======================================
  Hits         1428     1428           
  Misses        302      302           

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 0e02eae...9d58125. Read the comment docs.

@bigtimebuddy bigtimebuddy added the ✅ Ready To Merge Helpful when issues are in the queue waiting to get merged. This means the PR is completed and has t label May 20, 2020
.gitignore Outdated Show resolved Hide resolved
Co-authored-by: Matt Karl <matt@mattkarl.com>
@ShukantPal
Copy link
Member Author

Sharp 👀 @bigtimebuddy

@bigtimebuddy bigtimebuddy merged commit d07aeb5 into dev May 24, 2020
@bigtimebuddy bigtimebuddy deleted the improve/gitignore branch May 24, 2020 23:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✅ Ready To Merge Helpful when issues are in the queue waiting to get merged. This means the PR is completed and has t
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants