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

add/run: improve "already tracked" message #3678

Closed
efiop opened this issue Apr 24, 2020 · 1 comment · Fixed by #3808
Closed

add/run: improve "already tracked" message #3678

efiop opened this issue Apr 24, 2020 · 1 comment · Fixed by #3808
Labels
enhancement Enhances DVC good first issue help wanted p2-medium Medium priority, should be done, but less important ui user interface / interaction

Comments

@efiop
Copy link
Contributor

efiop commented Apr 24, 2020

As noted by @ammarasmro

#3561 (comment)

our ERROR: output 'models' is already tracked by SCM (e.g. Git) could be improved.

Looks like we could recommend

git rm -r --cached path
git commit -m "stop tracking path"

as a solution. If the file/dir was accidentally added to git and is really big, then we could also recommend https://rtyley.github.io/bfg-repo-cleaner/ or something like that to remove it from the history.

@triage-new-issues triage-new-issues bot added the triage Needs to be triaged label Apr 24, 2020
@efiop efiop added enhancement Enhances DVC good first issue help wanted ui user interface / interaction labels Apr 24, 2020
@triage-new-issues triage-new-issues bot removed the triage Needs to be triaged label Apr 24, 2020
@efiop efiop added p3-nice-to-have It should be done this or next sprint p2-medium Medium priority, should be done, but less important and removed p3-nice-to-have It should be done this or next sprint labels Apr 24, 2020
PuneethaPai added a commit to PuneethaPai/dvc that referenced this issue May 17, 2020
Fixes: iterative#3678
When file has been already tracked under git, OutputAlreadyTrackedError gets raised.
This commits add git command suggestions to help user to stop tracking file from git,
so in future commits they can use DVC for the file.
@efiop
Copy link
Contributor Author

efiop commented May 21, 2020

Let's not recommend bfg for now, as it is too specific.

efiop pushed a commit that referenced this issue May 21, 2020
…es (#3808)

Fixes: #3678
When file has been already tracked under git, OutputAlreadyTrackedError gets raised.
This commits add git command suggestions to help user to stop tracking file from git,
so in future commits they can use DVC for the file.

Co-authored-by: Puneetha Pai <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Enhances DVC good first issue help wanted p2-medium Medium priority, should be done, but less important ui user interface / interaction
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant