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.

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

Use remote plugins for build #223

Merged
merged 4 commits into from
Nov 13, 2024
Merged

Use remote plugins for build #223

merged 4 commits into from
Nov 13, 2024

Conversation

tstirrat15
Copy link
Contributor

@tstirrat15 tstirrat15 commented Nov 1, 2024 •
edited
Loading

Description

This makes buf generation use remote plugins so we don't have to fiddle too much.

Change

  • Update API update github actions to take advantage of buf remote plugins
  • Regenerate lib against api version 1.38
  • Update buf.gen.yaml to be a normal yaml file

Testing

Review. See that things go green.

@tstirrat15 tstirrat15 requested a review from a team as a code owner November 1, 2024 23:09
@tstirrat15 tstirrat15 force-pushed the use-remote-plugins-and-bump-version branch from 497aa59 to cb13b75 Compare November 12, 2024 23:58
@tstirrat15 tstirrat15 changed the title Bump API version and use remote plugins for build Use remote plugins for build Nov 12, 2024
@vroldanbet vroldanbet added this pull request to the merge queue Nov 13, 2024
Merged via the queue into main with commit 4d44ce1 Nov 13, 2024
13 checks passed
@vroldanbet vroldanbet deleted the use-remote-plugins-and-bump-version branch November 13, 2024 09:45
@github-actions github-actions bot locked and limited conversation to collaborators Nov 13, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@vroldanbet vroldanbet vroldanbet approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@tstirrat15 @vroldanbet