Skip to content
This repository was archived by the owner on Jun 3, 2026. It is now read-only.

Discard dispatch API response body#3

Merged
jameslamb merged 1 commit into
rapidsai:masterfrom
bdice:fix/jq-workflow-id-integer-overflow
Dec 19, 2025
Merged

Discard dispatch API response body#3
jameslamb merged 1 commit into
rapidsai:masterfrom
bdice:fix/jq-workflow-id-integer-overflow

Conversation

@bdice

@bdice bdice commented Dec 19, 2025

Copy link
Copy Markdown

Backporting convictional#109 to our fork.

Resolves issues discussed in https://github.com/orgs/community/discussions/9752#discussioncomment-15294117.

That discussion mentions the API break will be rolled back but GitHub may still make this change in January 2026. This change should make our action work either way, in my understanding.

@gforsyth gforsyth 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.

I'll let others take a look, too, but this seems like a reasonable fix, will protect us from future changes, and isn't overly invasive.

@jameslamb jameslamb left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I agree, this looks like a relatively future-proof fix.

@rockhowse
rockhowse self-requested a review December 19, 2025 16:53

@rockhowse rockhowse 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.

After reviewing the following

I am ok with this solution if it unblocks nightlies and CCCL 3.2 work.

@bdice

bdice commented Dec 19, 2025

Copy link
Copy Markdown
Author

/merge

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants