Skip to content

ci(proto): automatically publish proto files#571

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

ci(proto): automatically publish proto files#571
aelesbao merged 3 commits into
mainfrom
sol-280-automatically-publish-the-proto-files

Conversation

@aelesbao

@aelesbao aelesbao commented May 16, 2024

Copy link
Copy Markdown
Contributor

Automatically publish the protocol's ProtoBuf files to the Buf Schema Registry

@aelesbao aelesbao requested review from a team, philabsbot and spoo-bar and removed request for philabsbot May 16, 2024 12:18
@aelesbao aelesbao force-pushed the sol-280-automatically-publish-the-proto-files branch from 946a58e to 0068541 Compare May 16, 2024 12:20

@shahbazn shahbazn 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

@archway-bot archway-bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

lgtm

@aelesbao aelesbao requested review from zanicar and removed request for spoo-bar May 16, 2024 13: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 68f8e29 into main May 16, 2024
@aelesbao aelesbao deleted the sol-280-automatically-publish-the-proto-files branch May 16, 2024 15:31
spoo-bar pushed a commit that referenced this pull request May 21, 2024
* ci(proto): automatically publish proto files

* chore: add changes

* ci(proto): also push to proto files on tag
@spoo-bar

Copy link
Copy Markdown
Contributor

@aelesbao I merged this PR to the v7 release branch and tagged v7.0.1. Its failing, can you take a look https://github.com/archway-network/archway/actions/runs/9173751330/job/25223098662

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.

5 participants