-
Notifications
You must be signed in to change notification settings - Fork 31
Require more jobs in merge queue #102
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.
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
Conversation
|
Thank you for the review. It seems to fail at the part where line-openapi is injected into other SDK repositories. |
| run: | | ||
| cd line-openapi | ||
| git remote add pr-source $PR_REPO_URL | ||
| git fetch pr-source $PR_REF | ||
| git checkout FETCH_HEAD | ||
| env: | ||
| PR_REPO_URL: ${{ github.event.pull_request.head.repo.clone_url }} | ||
| PR_REF: ${{ github.event.pull_request.head.ref }} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a way to make it work only when pull request. This change fixes the checkout way so that it always works as expected.
JAVAYou can check generated code in java |
PHPYou can check generated code in php |
NODEJSYou can check generated code in nodejs |
GOYou can check generated code in go |
PYTHONYou can check generated code in python |
To re-enable the merge queue, this change adds merge_group to several jobs.
original: line/line-bot-sdk-java#1592
Additionally, by using actions/checkout, this change modifies how to inject the latest openapi schema so that the bot SDK tests always run with the latest line-openapi, even outside of PRs. This will be a safer approach.