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

Update protos to buf tag 6bf58783fdc3ed33da39a728e2538c1cb58da34e #27

Merged
merged 1 commit into from
Sep 7, 2022

Conversation

samkim
Copy link
Member

@samkim samkim commented Sep 6, 2022

No description provided.

@samkim samkim requested a review from a team September 6, 2022 21:44
@@ -23,7 +23,7 @@ jobs:
java-version: "${{ matrix.java }}"
- uses: "bufbuild/buf-setup-action@v1.7.0"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we instead use our centralized buf action?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The shared action also generates assuming a buf.gen.yaml which isn't used in the Java client.

@samkim samkim merged commit 196a652 into authzed:main Sep 7, 2022
@github-actions github-actions bot locked and limited conversation to collaborators Sep 7, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants