Skip to content

add cutover service & protos#319

Open
insipx wants to merge 2 commits intomainfrom
push-oysumkopzwzu
Open

add cutover service & protos#319
insipx wants to merge 2 commits intomainfrom
push-oysumkopzwzu

Conversation

@insipx
Copy link
Contributor

@insipx insipx commented Feb 4, 2026

No description provided.

@insipx insipx requested a review from a team as a code owner February 4, 2026 18:12
@macroscopeapp
Copy link

macroscopeapp bot commented Feb 4, 2026

Add xmtp.migration.api.v1.D14nMigrationApi.FetchD14nCutover endpoint and configure Buf publishing on push in buf.yml

Define xmtp.migration.api.v1 with D14nMigrationApi.FetchD14nCutover returning timestamp_ns and update Buf workflow to install via bufbuild/buf-action@v1 and run buf push only on push events.

📍Where to Start

Start with the service and RPC in migration.proto.


Macroscope summarized 31d8397.

@insipx insipx force-pushed the push-oysumkopzwzu branch from 3e1c77c to 10ab2a3 Compare February 4, 2026 18:20
@insipx insipx force-pushed the push-oysumkopzwzu branch from 10ab2a3 to 8876c0e Compare February 4, 2026 18:21
@insipx insipx force-pushed the push-oysumkopzwzu branch from 8876c0e to 31d8397 Compare February 4, 2026 18:29
token: ${{ secrets.BUF_TOKEN }}
setup_only: true # Disables default behavior.
- if: ${{ github.event_name == 'push' }}
run: buf push --error-format github-actions --create --git-metadata
Copy link

Choose a reason for hiding this comment

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

🟠 High

workflows/buf.yml:19 The previous workflow used input: proto. Without specifying the directory, buf push will fail because buf.yaml is in proto/, not the repository root. Consider adding proto to the command.

Suggested change
run: buf push --error-format github-actions --create --git-metadata
run: buf push proto --error-format github-actions --create --git-metadata

🚀 Want me to fix this? Reply ex: "fix it for me".

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.

1 participant