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

Bump API version and fix automatic bumping #136

Merged
merged 11 commits into from
Nov 4, 2024
Merged

Conversation

tstirrat15
Copy link
Contributor

@tstirrat15 tstirrat15 commented Nov 1, 2024

Description

Gets the lib on 1.38. Also changes how generation is done.

Changes

See annotations

Testing

See that things go green.

@tstirrat15 tstirrat15 requested a review from a team November 1, 2024 21:55
@tstirrat15 tstirrat15 changed the title Fix bumping of release version Bump API version and fix automatic bumping Nov 1, 2024
Copy link
Contributor Author

@tstirrat15 tstirrat15 left a comment

Choose a reason for hiding this comment

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

See comments

.github/workflows/automatic-api-update.yaml Show resolved Hide resolved
.github/workflows/automatic-api-update.yaml Show resolved Hide resolved
.github/workflows/automatic-api-update.yaml Show resolved Hide resolved
.github/workflows/automatic-api-update.yaml Show resolved Hide resolved
.github/workflows/manual-api-update.yaml Show resolved Hide resolved
buf.gen.yaml Show resolved Hide resolved
buf.gen.yaml Show resolved Hide resolved
lib/authzed/api/v1/error_reason_pb.rb Show resolved Hide resolved
Copy link
Contributor

@vroldanbet vroldanbet left a comment

Choose a reason for hiding this comment

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

LGTM! This PR supersedes the automatically opened #137, right?

@@ -1,9 +1,9 @@
#!/usr/bin/env -S buf generate --template
Copy link
Contributor

Choose a reason for hiding this comment

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

tested locally with buf generate and got the same output 👍🏻

@vroldanbet vroldanbet mentioned this pull request Nov 4, 2024
@tstirrat15 tstirrat15 merged commit bf9d4a0 into main Nov 4, 2024
8 checks passed
@tstirrat15 tstirrat15 deleted the fix-release-process branch November 4, 2024 15:36
@github-actions github-actions bot locked and limited conversation to collaborators Nov 4, 2024
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