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: add -- to git add command to denote pathspec starting #821

Merged
merged 1 commit into from Mar 31, 2020

Conversation

iiroj
Copy link
Member

@iiroj iiroj commented Mar 31, 2020

Without the -- denoter some paths might be interpreted as flags.

Fixes #820

Without the -- denoter some paths might be interpreted as flags
@codecov
Copy link

codecov bot commented Mar 31, 2020

Codecov Report

Merging #821 into master will not change coverage by %.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##            master      #821   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           13        13           
  Lines          514       514           
  Branches       110       110           
=========================================
  Hits           514       514           
Impacted Files Coverage Δ
lib/gitWorkflow.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 8abb09f...e5e97de. Read the comment docs.

@yoyo837
Copy link

yoyo837 commented Mar 31, 2020

Thanks for the catch!

@iiroj iiroj merged commit 226ccdb into master Mar 31, 2020
@iiroj iiroj deleted the fix-git-add branch March 31, 2020 13:53
@okonet
Copy link
Collaborator

okonet commented Mar 31, 2020

🎉 This PR is included in version 10.1.1 🎉

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
Labels
Development

Successfully merging this pull request may close these issues.

Applying modifications... [failed]
3 participants