Repository navigation
ci(workflow): ⚙️ refactor pages workflow concurrency and artifact‑upload position - #8
Merged
Merged
Conversation
…oad position - Rename concurrency group to `pages-deploy`, set `cancel-in-progress: true` Abort outdated running jobs automatically, prevent deployment race‑condition and artifact overwriting - Move `actions/upload-pages-artifact` step from build job to deploy job PR stage completes compilation verification inside ephemeral runner without uploading pages artifact - Strengthen deploy‑job condition: only push to main branch or manual dispatch triggers formal deployment Pull‑request will skip deploy job entirely - Keep original build pipeline: build Nuxt main site, build VitePress docs and merge dist files into `.output/public/docs` Reduce useless artifacts generated from PR builds, mitigate concurrent deployment coverage risk. No related issues.
Skixkk
commented
Aug 16, 2026
Skixkk
left a comment
Skixkk
left a comment
Collaborator
Author
There was a problem hiding this comment.
Workflow concurrency rule and artifact upload position updated.
PR will not produce pages artifacts, deployment only triggered on main push or manual run.
Checklist:
- Concurrency group name corrected
- cancel‑in‑progress set to true
- Artifact upload moved into deploy job
- Deploy job condition works correctly
LGTM
Skixkk
commented
Aug 16, 2026
| @@ -49,20 +53,21 @@ jobs: | |||
| mkdir -p .output/public/docs | |||
| cp -r docs/.vitepress/dist/* .output/public/docs/ | |||
Collaborator
Author
There was a problem hiding this comment.
add debug directory‑listing step after docs copy
- Insert debug shell step after vitepress dist copy operation - Execute `ls -l -a` against `.output/public` and `.output/public/docs` - Print directory contents to workflow log, convenient for troubleshooting missing docs files - Keep all existing concurrency、artifact‑upload、deploy‑condition rules unchanged Add visibility of merged output folder for deployment debugging, zero functional change. No related issues.
Skixkk
commented
Aug 16, 2026
Skixkk
left a comment
Skixkk
left a comment
Collaborator
Author
There was a problem hiding this comment.
added debug directory‑listing step after docs copy
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Changes
pages‑deployand enablecancel‑in‑progress: trueAutomatically cancel stale running jobs to avoid deployment conflict and artifact overwriting
PR builds run compilation check only, no pages artifact will be generated
Deployment executes only when pushing to main branch or manual workflow dispatch
.output/public/docsPurpose
Optimize CI/CD behavior, separate PR build verification and formal pages deployment.
Eliminate redundant artifacts and reduce race‑condition risk.
Impact