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.

Sign up for GitHub

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

Jump to bottom

Ignoring statuses of synched files under vendor when publishing #224

Open

khanism wants to merge 1 commit into helsing-ai:main from khanism:dirty_publish

khanism commented Feb 16, 2024

Regarding issue #200
This change adds some options to the check of the repository when publishing your Buffrs project.

When publishing, the statuses of the synched *.proto files under vendor are not included, avoiding the project being dirty as described in the issue


          Ignoring statuses of synched files under vendor when publishing

439dac1

khanism mentioned this pull request

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

Open

Contributor

mara-schulke commented Feb 21, 2024

Hi @khanism, thank you for contributing!

Have you seen the work on #217 / 78740b1 ?

I think this issue still exists, but the way we check statuses for git has been changed 😊

If you can rebase and update your implementation I am more than happy to merge this, tysm!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment