Skip to content

Conversation

@baywet
Copy link
Member

@baywet baywet commented Oct 29, 2025

fixes #202

@baywet baywet self-assigned this Oct 29, 2025
@baywet baywet force-pushed the ci/multiple-discussions branch from 83ec2e0 to 1f2af9f Compare October 29, 2025 14:34
@baywet baywet marked this pull request as ready for review October 29, 2025 14:37
@baywet baywet requested review from a team as code owners October 29, 2025 14:37
@baywet baywet enabled auto-merge October 29, 2025 14:38
@baywet
Copy link
Member Author

baywet commented Oct 29, 2025

This does multiple things:

  • Switches to using gh CLI instead of octokit to execute the GraphQL queries. I suspect ocktokit retries in some very specific cases, but the task does seem to be logging retrials. The CLI should be.
  • Adds a step to check whether the agenda item already exsits.

https://github.com/OAI/Overlay-Specification/actions/runs/18911572019/job/53983471269

Unfortunately, the discussions are not available in the GitHub CLI directly, which would make all of that simpler. And the extensions are disappointing to say the least. Additionally, the discussions don't seem to be available with REST endpoints for some reason.
The multiple creations could very well be a fluke of the backend graphQL resolver, in which case I'm afraid there won't be much of a way to solve it on our end. But these changes should add a bit more "safety" to this automation.

@lornajane
Copy link
Contributor

I think this is why we stuck with issues in the main OpenAPI project! Discussions are a nice idea but they're not very linear and they're less well supported. I'll review when I can (thank you for the explanation of the changes, that helps a lot!)

@baywet
Copy link
Member Author

baywet commented Oct 29, 2025

yeah from a documentation, API, and CLI perspective, it looks like they stopped the work midway through... For example: why is there a REST API for teams discussions (never used those), but not for repository discussions?

@baywet baywet merged commit 3b3f3e5 into main Oct 29, 2025
4 checks passed
@baywet baywet deleted the ci/multiple-discussions branch October 29, 2025 15:06
@baywet baywet mentioned this pull request Nov 4, 2025
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.

Agenda workflow created three discussions for the same date

4 participants