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

buffrs publish refuses to publish when the proto/vendor/ is dirty #200

Open
qsantos opened this issue Dec 15, 2023 · 2 comments
Open

buffrs publish refuses to publish when the proto/vendor/ is dirty #200

qsantos opened this issue Dec 15, 2023 · 2 comments
Labels
complexity::low Issues or ideas with a low implementation cost component::cli Everything related to the buffrs cli priority::medium This is not urgent, but we should do this. type::fix

Comments

@qsantos
Copy link
Contributor

qsantos commented Dec 15, 2023

When publishing, Buffrs populates the proto/vendor/ directory. If one attempts to publish again, Buffrs then complains that the repository is dirty. I think Buffrs should either:

  • clear proto/vendor/ before checking that the repository is clean
  • ignore proto/vendor/ when checking that the repository is clean
@qsantos qsantos changed the title buffrs publish refuses to publishes when the proto/vendor/ is dirty buffrs publish refuses to publish when the proto/vendor/ is dirty Dec 27, 2023
@khanism
Copy link

khanism commented Feb 16, 2024

I'm not sure if this was intended and if it is wanted to ignore/filter the messages that lead to buffrs complaining about dirty proto files under vendor. Anyhow since this was a quick one: #224

The PR does only include dirty statuses of the files actually being packaged. Should one wanted to add a subdirectory under proto (e.g. submodule that contains additional *.proto files) this solution wouldn't cover that but I'm not sure how much this is relevant as of now.

@mara-schulke
Copy link
Contributor

I think we should get #224 over the line!

@mara-schulke mara-schulke added complexity::low Issues or ideas with a low implementation cost component::cli Everything related to the buffrs cli priority::medium This is not urgent, but we should do this. type::fix labels Jul 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
complexity::low Issues or ideas with a low implementation cost component::cli Everything related to the buffrs cli priority::medium This is not urgent, but we should do this. type::fix
Projects
None yet
Development

No branches or pull requests

3 participants