Skip to content

feat: add automated proto generation workflow - #4931

Merged
cuixq merged 8 commits into
google:masterfrom
cuixq:proto
Mar 3, 2026
Merged

cuixq merged 8 commits into
google:masterfrom
cuixq:proto

Conversation

@cuixq

@cuixq cuixq commented Mar 3, 2026 •

Copy link
Copy Markdown
Contributor

#4233

This PR Adds a GitHub Action to automatically regenerate Python and Go code when proto files are updated.

  • The workflow runs on merges to master affecting gcp/api/v1/osv_service_v1.proto, osv/importfinding.proto, or vulnerability.proto in osv-schema submodule.
  • The workflow also sets up the environments needed for proto generation including Python (Poetrty) and Go, protoc and necessary plugins.
  • As an outcome, the workflow makes a pull request with the regenerated code for review.

@cuixq

cuixq commented Mar 3, 2026

Copy link
Copy Markdown
Contributor Author

/gemini review

@gemini-code-assist

Copy link
Copy Markdown
Contributor

Note

Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported.

@cuixq
cuixq requested a review from another-rex March 3, 2026 05:12
@cuixq
cuixq marked this pull request as ready for review March 3, 2026 05:12
@cuixq
cuixq merged commit ca172bc into google:master Mar 3, 2026
21 checks passed
@cuixq
cuixq deleted the proto branch March 3, 2026 23:32
tymzd pushed a commit to tymzd/osv.dev that referenced this pull request Apr 13, 2026
google#4233

This PR Adds a GitHub Action to automatically regenerate Python and Go
code when proto files are updated.
- The workflow runs on merges to master affecting
`gcp/api/v1/osv_service_v1.proto`, `osv/importfinding.proto`, or
`vulnerability.proto` in osv-schema submodule.
- The workflow also sets up the environments needed for proto generation
including Python (Poetrty) and Go, protoc and necessary plugins.
- As an outcome, the workflow makes a pull request with the regenerated
code for review.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants