Merge docs-dev and docs-release workflows into single docs.yml#819
Merged
Conversation
- Unified workflow handles both dev (push to master) and release builds - Fixed 'git add .' bug that accidentally committed stray files (like site/) - Now uses 'git add "$VERSION" latest' to only stage intended directories - Dev builds use --force push, release builds don't - Codespaces link pinning only runs for releases - Consistent artifact handling (move to /tmp before checkout)
✅ Deploy Preview for nextflow-training ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Contributor
|
Nextflow linting complete! ❌ 1 files had 1 errors 💡 Tip: Click filename locations to go directly to that code. View all 1 issues
View formatting changes
|
Allows republishing old versions without overwriting the 'latest' alias. When unchecked, only deploys to the version directory without updating 'latest'.
- Add .github/scripts/deploy_docs.py with all deployment logic - Simplify docs.yml to ~70 lines (was ~170 lines combined) - Uses uv for dependency management (gitpython) - Commands: discover-languages, get-version-info, deploy
This reverts commit 3ac2de6.
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.
Attempt to simplify the deployment logic and collapse into a single automation flow, so that we don't have issues like we did earlier today where a bug was fixed in one actions workflow but not the other.
Note that GitHub Pages is complaining about the size of our deployment. I investigated this a little and part of the reason is that we have a massive
site/directory there which is a build artefact and not actually used. This PR should avoid accidentally committing such build artefacts - once merged we can remove that from thegh-pagesbranch.Summary
docs-dev.ymlanddocs-release.ymlinto singledocs.ymlworkflowgit add .bug that accidentally committed stray files (caused the 213MBsite/directory on gh-pages)git add "$VERSION" latestto only stage intended directoriesChanges
docs-dev.yml+docs-release.ymldocs.ymlgit add .(dangerous)git add "$VERSION" latest(safe)How it works
0.devwith aliasdevelopment2.8.1) with aliaslatest, pins Codespaces linksRelated
site/directory (213MB) on gh-pages