-
Notifications
You must be signed in to change notification settings - Fork 153
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
Fix the workflow publishes quickstart manifests #5266
Conversation
Signed-off-by: Shinnosuke Sawada-Dazai <[email protected]>
Signed-off-by: Shinnosuke Sawada-Dazai <[email protected]>
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.
Nice catch 👍
I found a better solution. I'll change the actions/checkout arguments, not the create-pull-request arguments. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #5266 +/- ##
==========================================
- Coverage 24.09% 24.08% -0.01%
==========================================
Files 441 441
Lines 47099 47099
==========================================
- Hits 11348 11345 -3
- Misses 34847 34850 +3
Partials 904 904 ☔ View full report in Codecov by Sentry. |
Signed-off-by: Shinnosuke Sawada-Dazai <[email protected]>
Signed-off-by: Shinnosuke Sawada-Dazai <[email protected]>
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.
Looks better 👍
* Add pull-requests permission to the workflow Signed-off-by: Shinnosuke Sawada-Dazai <[email protected]> * Set the base branch to the default branch of the repository Signed-off-by: Shinnosuke Sawada-Dazai <[email protected]> * Use `github.ref_name` to determine the version of manifests Signed-off-by: Shinnosuke Sawada-Dazai <[email protected]> * Checkout master branch to make PR to master branch Signed-off-by: Shinnosuke Sawada-Dazai <[email protected]> --------- Signed-off-by: Shinnosuke Sawada-Dazai <[email protected]> Signed-off-by: pipecd-bot <[email protected]>
* Fix the workflow publishes quickstart manifests (#5266) * Add pull-requests permission to the workflow Signed-off-by: Shinnosuke Sawada-Dazai <[email protected]> * Set the base branch to the default branch of the repository Signed-off-by: Shinnosuke Sawada-Dazai <[email protected]> * Use `github.ref_name` to determine the version of manifests Signed-off-by: Shinnosuke Sawada-Dazai <[email protected]> * Checkout master branch to make PR to master branch Signed-off-by: Shinnosuke Sawada-Dazai <[email protected]> --------- Signed-off-by: Shinnosuke Sawada-Dazai <[email protected]> Signed-off-by: pipecd-bot <[email protected]> * Remove pipectl quickstart command (#5268) * Remove pipectl quickstart command Signed-off-by: khanhtc1202 <[email protected]> * Remove the quickstart from docs Signed-off-by: khanhtc1202 <[email protected]> --------- Signed-off-by: khanhtc1202 <[email protected]> Signed-off-by: pipecd-bot <[email protected]> * Enabled to configure the interval of livestate store for Lambda (#5269) * add liveStateInterval for Lambda Signed-off-by: t-kikuc <[email protected]> * add docs of liveStateInterval Signed-off-by: t-kikuc <[email protected]> * fix a test Signed-off-by: t-kikuc <[email protected]> * Rename to 'awsAPIPollingInterval' Signed-off-by: t-kikuc <[email protected]> --------- Signed-off-by: t-kikuc <[email protected]> Signed-off-by: pipecd-bot <[email protected]> * Update RELEASE to v0.49.2 and sync docs of v0.49.x (#5272) Signed-off-by: t-kikuc <[email protected]> Signed-off-by: pipecd-bot <[email protected]> --------- Signed-off-by: Shinnosuke Sawada-Dazai <[email protected]> Signed-off-by: pipecd-bot <[email protected]> Signed-off-by: khanhtc1202 <[email protected]> Signed-off-by: t-kikuc <[email protected]> Co-authored-by: Shinnosuke Sawada-Dazai <[email protected]> Co-authored-by: Khanh Tran <[email protected]> Co-authored-by: Tetsuya Kikuchi <[email protected]>
What this PR does / why we need it:
ref; https://github.com/peter-evans/create-pull-request?tab=readme-ov-file#token
ref; https://github.com/pipe-cd/pipecd/actions/runs/11229319582/job/31214779847#step:6:52
Which issue(s) this PR fixes:
Follow PR #5260
Does this PR introduce a user-facing change?: