Skip to content

gc: disallow --remote without other flags#9443

Merged
daavoo merged 1 commit into
mainfrom
gc-remote-arg-error
May 16, 2023
Merged

gc: disallow --remote without other flags#9443
daavoo merged 1 commit into
mainfrom
gc-remote-arg-error

Conversation

@dberenbaum
Copy link
Copy Markdown
Contributor

Fixes #6970

@dberenbaum dberenbaum marked this pull request as ready for review May 11, 2023 16:25
@codecov
Copy link
Copy Markdown

codecov Bot commented May 11, 2023

Codecov Report

Patch coverage: 81.81% and project coverage change: -0.01 ⚠️

Comparison is base (1e538ed) 91.60% compared to head (2f30bbc) 91.60%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #9443      +/-   ##
==========================================
- Coverage   91.60%   91.60%   -0.01%     
==========================================
  Files         489      489              
  Lines       38167    38178      +11     
  Branches     5469     5472       +3     
==========================================
+ Hits        34963    34972       +9     
- Misses       2639     2640       +1     
- Partials      565      566       +1     
Impacted Files Coverage Δ
dvc/commands/gc.py 83.60% <ø> (ø)
dvc/repo/gc.py 96.92% <33.33%> (-3.08%) ⬇️
tests/unit/command/test_gc.py 100.00% <100.00%> (ø)

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@daavoo daavoo enabled auto-merge (rebase) May 16, 2023 14:15
@daavoo daavoo force-pushed the gc-remote-arg-error branch from 3e8fb90 to 2f30bbc Compare May 16, 2023 14:15
@daavoo daavoo merged commit f888544 into main May 16, 2023
@daavoo daavoo deleted the gc-remote-arg-error branch May 16, 2023 15:18
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.

gc: clarify behavior of cloud/remote options

2 participants