Skip to content

Conversation

@AndrewWinterman
Copy link
Contributor

Please read CONTRIBUTING.md for additional information on contributing to this repository!

What this PR does / why we need it

I'd like to test out buf in a new service (tombstone), but the existing version of buf is so old it doesn't support the version2 configuration files.

This PR upgrades the buf version.

I am still testing this out, but I think it should be safe since buf still supports v1 yaml schemas.

Jira ID

[XX-XX]

Notes for your reviewers

@AndrewWinterman AndrewWinterman requested a review from a team as a code owner September 24, 2025 18:25
Copy link
Member

@malept malept left a comment

Choose a reason for hiding this comment

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

I already discussed this with Andrew, but this needs an ecosystem regression test before we can approve.

@malept malept changed the title fix: buf version to 1.57.2 fix: upgrade buf to 1.57.2 Sep 24, 2025
@AndrewWinterman AndrewWinterman changed the title fix: upgrade buf to 1.57.2 fix: buf version to 1.57.2 Oct 9, 2025
@getoutreach-ci-1
Copy link
Contributor

Link to code coverage report (posted by coverbot 🤖)

@AndrewWinterman
Copy link
Contributor Author

AndrewWinterman commented Oct 13, 2025

Run is here https://app.circleci.com/pipelines/github/getoutreach/crucible-runs/109/workflows/08b4929e-ac2d-469a-893b-673078e6e1cd

Only failures as of posting are repo not found

https://app.circleci.com/pipelines/github/getoutreach/crucible-runs/109/workflows/08b4929e-ac2d-469a-893b-673078e6e1cd/jobs/3864:

{"level":"error","repo":"msftspeechapi","dir":"/tmp/crucible-test-1777529767/msftspeechapi","url":"[email protected]:getoutreach/msftspeechapi","error":"repository not found","time":"2025-10-13T09:57:38Z","message":"FAILED TO CLONE REPO"}

https://app.circleci.com/pipelines/github/getoutreach/crucible-runs/109/workflows/08b4929e-ac2d-469a-893b-673078e6e1cd/jobs/3966

{"level":"error","repo":"ds-cds-golang-forecast-service","dir":"/tmp/crucible-test-2532380528/ds-cds-golang-forecast-service","url":"[email protected]:getoutreach/ds-cds-golang-forecast-service","error":"repository not found","time":"2025-10-13T09:57:20Z","message":"FAILED TO CLONE REPO"}

@malept malept changed the title fix: buf version to 1.57.2 fix: buf version to 1.60.0 Nov 25, 2025
@malept malept changed the title fix: buf version to 1.60.0 fix(linters): upgrade buf to 1.60.0 Nov 25, 2025
gojq: v0.12.17

# protobuf formatters/plugins/tools/etc
buf: 1.8.0 # formatter
Copy link
Member

Choose a reason for hiding this comment

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

Deleted the comment because it will be more than a formatter starting with #1009

@malept malept merged commit 220af00 into main Nov 26, 2025
8 checks passed
@malept malept deleted the fix-buf-version branch November 26, 2025 00:07
@getoutreach-ci-2
Copy link

🎉 This PR is included in version 2.34.6-rc.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants