Skip to content

ci(proto): use buf 1.31 to push files#572

Merged
aelesbao merged 7 commits into
mainfrom
sol-280-automatically-publish-the-proto-files
May 23, 2024
Merged

ci(proto): use buf 1.31 to push files#572
aelesbao merged 7 commits into
mainfrom
sol-280-automatically-publish-the-proto-files

Conversation

@aelesbao

@aelesbao aelesbao commented May 21, 2024

Copy link
Copy Markdown
Contributor

The buf CLI v1.32 seems to have an undocumented breaking change and won't accept the buf.yaml file structure defined on v1.

This is possibly a bug on their side since it's mentioned in the changelog that the CLI is backwards compatible with v1 projects.

@aelesbao aelesbao requested review from a team and philabsbot May 21, 2024 12:36
@aelesbao aelesbao changed the title fix(proto): install buf v1.32 to push proto files ci(proto): use buf 1.31 to push files May 21, 2024
@aelesbao aelesbao requested review from a team and spoo-bar May 21, 2024 17:35
@aelesbao aelesbao requested review from zanicar and removed request for philabsbot May 21, 2024 17:40

@zanicar zanicar left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@aelesbao aelesbao merged commit c5dcaa3 into main May 23, 2024
@aelesbao aelesbao deleted the sol-280-automatically-publish-the-proto-files branch May 23, 2024 14:21
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