Skip to content

log: show modifying if entry already exists in dvcfile#3978

Merged
efiop merged 1 commit into
treeverse:masterfrom
skshetry:fix-log-dvcfile-modif
Jun 8, 2020
Merged

log: show modifying if entry already exists in dvcfile#3978
efiop merged 1 commit into
treeverse:masterfrom
skshetry:fix-log-dvcfile-modif

Conversation

@skshetry
Copy link
Copy Markdown
Collaborator

@skshetry skshetry commented Jun 8, 2020

Related #3933

  • ❗ I have followed the Contributing to DVC checklist.

  • 📖 If this PR requires documentation updates, I have created a separate PR (or issue, at least) in dvc.org and linked it here. If the CLI API is changed, I have updated tab completion scripts.

  • ❌ I will check DeepSource, CodeClimate, and other sanity checks below. (We consider them recommendatory and don't expect everything to be addressed. Please fix things that actually improve code or fix bugs.)

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

@skshetry skshetry added bug Did we break something? c1-quick-fix labels Jun 8, 2020
@skshetry skshetry requested a review from efiop June 8, 2020 13:20
@skshetry skshetry self-assigned this Jun 8, 2020
@efiop efiop merged commit 215ea49 into treeverse:master Jun 8, 2020
@skshetry skshetry deleted the fix-log-dvcfile-modif branch June 8, 2020 15:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Did we break something?

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants