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

Raise an error if ther are upstream changes before running TAF metadata, targets commands #225

Open
n-dusan opened this issue Apr 2, 2022 · 2 comments
Assignees
Labels

Comments

@n-dusan
Copy link
Contributor

n-dusan commented Apr 2, 2022

We shouldn't update metadata if the repository isn't up to date.

@n-dusan n-dusan self-assigned this Jul 5, 2022
@renatav
Copy link
Collaborator

renatav commented May 16, 2024

Do we want to automatically update or raise an error if the repository is not up to date?

@renatav renatav moved this to Backlog in TAF Planning Jun 30, 2024
@renatav renatav added the good first issue Good for newcomers label Oct 3, 2024
@renatav renatav self-assigned this Oct 4, 2024
@renatav renatav moved this from Backlog to Todo in TAF Planning Oct 4, 2024
@renatav renatav changed the title Run updater before TAF metadata, targets commands Raise an error if ther are upstream changes before running TAF metadata, targets commands Nov 19, 2024
@renatav
Copy link
Collaborator

renatav commented Nov 19, 2024

My final suggestion is to simply raise an error if there are upstream changes, instead of automatically running the updater. The user might want to take a look at those changes before updating anything. I'll integrate this into one of reworks

@renatav renatav moved this from Todo to In Progress in TAF Planning Nov 19, 2024
@renatav renatav moved this from In Progress to Todo in TAF Planning Nov 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Todo
Development

No branches or pull requests

2 participants