Rename commit
cmd --no-commit options to cache
and --no-cache
#5948
jorgeorpinel
started this conversation in
New Features & Ideas
Replies: 1 comment
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
After reviewing and simplifying the
dvc commit
cmd ref (in iterative/dvc.org/pull/1978) I realized it can be confusing to call this "committing" since we're mainly talking about caching (or not) data. Yes, it can also change hash values in DVC files but that's kind of an edge case given the main use for this: Data/code dev process whereadd/repro/run --no-commit
are used until ready, and finished by acommit
— the hash values are already saves in the DVC files byadd/repro/run
.And we also have
run --metrics/plots-no-cache
, sono-cache
is more consistent with those.Finally, changing
dvc commit
todvc cache
and--no-commit
to--no-cache
would also help avoid confusions related to the Git analogies we use. Specifically, it will help clarify that DVC does not perform the actual versioning, but that Git does, through commits. Currently we need to have the following clarification near the top of the ref:Maybe a change for 2.x? And it doesn't have to break backward compatibility if the old names are kept as aliases.
Beta Was this translation helpful? Give feedback.
All reactions