-
Notifications
You must be signed in to change notification settings - Fork 0
面接用リリースの版管理とテンプレート事前確認を整備 #47
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
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -34,27 +34,11 @@ cp -R golang "$work" | |
|
|
||
| Kick the [release action](https://github.com/folio-sec/wrap-api-example/actions/workflows/release.yml) manually, then release. | ||
|
|
||
| - `<language>-template.zip` | ||
| - `<language>.patch` | ||
| - `<language>.zip` | ||
| Each release has a sequential tag (`v1`, `v2`, ...). Its assets are: | ||
|
|
||
| ## On Interview | ||
|
|
||
| - macOS / Linux / etc | ||
|
|
||
| ```sh | ||
| curl -fsSL "https://github.com/folio-sec/wrap-api-example/releases/download/<tag>/<language>.patch" | patch -p1 | ||
| ``` | ||
|
|
||
| Windows(PowerShell): | ||
|
|
||
| ```powershell | ||
| curl.exe -fsSL "https://github.com/folio-sec/wrap-api-example/releases/download/<tag>/<language>.patch" -o interview.patch | ||
| git apply interview.patch | ||
| Remove-Item interview.patch | ||
| ``` | ||
|
|
||
| Alternative, use full zip. | ||
| - `<language>-template-vN.zip` — share this before the interview | ||
| - `<language>-vN.patch` — apply this version's patch during the interview | ||
| - `<language>-vN.zip` — completed alternative if applying the patch is difficult | ||
|
Comment on lines
+39
to
+41
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win Restore the interview-day patch procedure. This asset list replaces the former patch-application instructions. Add steps to check 🤖 Prompt for AI Agents |
||
|
|
||
| ### DCO Sign-Off Methods | ||
|
|
||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -15,4 +15,8 @@ go build ./... | |
|
|
||
| # テスト実行 | ||
| go test ./... | ||
|
|
||
| # 事前コマンド確認 | ||
| command -v curl | ||
| command -v patch | ||
|
Comment on lines
+20
to
+21
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win 事前確認で
📍 Affects 8 files
🤖 Prompt for AI Agents |
||
| ``` | ||
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.
🩺 Stability & Availability | 🟠 Major | ⚡ Quick win
🔎 Supported by static analysis
🏁 Script executed:
sed -n '1,40p' .github/workflows/release.ymlRepository: folio-sec/wrap-api-example
Length of output: 849
🌐 Web query:
GitHub Actions workflow syntax concurrency queue key pending runs cancelled💡 Result:
<source_evidence>
Citations:
Queue pending manual release runs.
This workflow runs only through
workflow_dispatch. With the defaultqueue: single, a third dispatch cancels the existing pending run. If each dispatch must produce a release, setqueue: max. This retains up to 100 pending runs; additional runs are still canceled.Suggested fix
concurrency: group: interview-release + queue: max cancel-in-progress: false📝 Committable suggestion
🤖 Prompt for AI Agents