Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[CI] GitHub script v5 introduced breaking changes #36224

Closed
assignUser opened this issue Jun 21, 2023 · 4 comments · Fixed by #36225
Closed

[CI] GitHub script v5 introduced breaking changes #36224

assignUser opened this issue Jun 21, 2023 · 4 comments · Fixed by #36225
Assignees
@assignUser
Labels
Component: Continuous Integration Type: bug
Milestone
13.0.0

Comments

@assignUser
Copy link
Member

Describe the bug, including details regarding any error messages, version, and platform.

The rest methods are not in github. anymore but github.rest. e.g. github.rest.issues

Component(s)

Continuous Integration
This was referenced Jun 21, 2023
Merged
Merged
@assignUser
Copy link
Member Author

@kou here is a failing job: https://github.com/apache/arrow/actions/runs/5338737833/jobs/9676509316?pr=36222

@kou kou changed the title [CI] Github script v5 intorduced breaking changes [CI] GitHub script v5 intorduced breaking changes Jun 21, 2023
@kou
Copy link
Member

kou commented Jun 21, 2023

Thanks!

kou pushed a commit that referenced this issue Jun 21, 2023
@assignUser
GH-36224: [CI] Update rest api invocations in GitHub scripts (#36225)
2329a7b 
The rest api calls were moved into from `github.` to `github.rest.` so the workflows are failing after the update to gh script v6.

### Are these changes tested?

As these changes are in priviliged workflows testing is only possible after merge.

Closes #36224 

Authored-by: Jacob Wujciak-Jens <jacob@wujciak.de>
Signed-off-by: Sutou Kouhei <kou@clear-code.com>
@kou kou added this to the 13.0.0 milestone Jun 21, 2023
@kou
Copy link
Member

kou commented Jun 21, 2023

Issue resolved by pull request 36225
#36225

@assignUser
Copy link
Member Author

@kou I triggered the previously failing pr again and it works correctly now: #36222

@assignUser assignUser changed the title [CI] GitHub script v5 intorduced breaking changes [CI] GitHub script v5 introduced breaking changes Jul 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@assignUser assignUser

Labels
Component: Continuous Integration Type: bug
Projects
None yet
Milestone
13.0.0
Development

Successfully merging a pull request may close this issue.

GH-36224: [CI] Update rest api invocations in GitHub scripts assignUser/arrow
2 participants
@kou @assignUser