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

check-ignore: update help output [qa] #4358

Merged
merged 4 commits into from Aug 11, 2020
Merged

check-ignore: update help output [qa] #4358

merged 4 commits into from Aug 11, 2020

Conversation

jorgeorpinel
Copy link
Contributor

@jorgeorpinel jorgeorpinel commented Aug 7, 2020

per #4282 (review)
and #4282 (review)

Thank you for the contribution - we'll try to review it as soon as possible. 🙏

@jorgeorpinel jorgeorpinel changed the title check_ignore: update help output check-ignore: update help output Aug 7, 2020
@codecov
Copy link

codecov bot commented Aug 7, 2020

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #4358   +/-   ##
=======================================
  Coverage   90.92%   90.92%           
=======================================
  Files         178      178           
  Lines       12352    12352           
=======================================
  Hits        11231    11231           
  Misses       1121     1121           
Impacted Files Coverage Δ
dvc/command/check_ignore.py 97.14% <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 78458ee...500c4dc. Read the comment docs.

@efiop efiop merged commit 9cb32b5 into master Aug 11, 2020
@efiop efiop deleted the cmd/check_ignore branch August 11, 2020 08:12
@jorgeorpinel jorgeorpinel changed the title check-ignore: update help output check-ignore: update help output [qa] Aug 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants