| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
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
| Original file line number | Diff line number | Diff line change | ||||||
|---|---|---|---|---|---|---|---|---|
| Expand Up | @@ -12,12 +12,15 @@ jobs: | |||||||
| strategy: | ||||||||
| fail-fast: false | ||||||||
| matrix: | ||||||||
| branch: ["3.14", "3.13", "3.12"] | ||||||||
| python-version: ["3.15", "3.14", "3.13", "3.12"] | ||||||||
| include: | ||||||||
| - python-version: "3.15" | ||||||||
| branch: "main" | ||||||||
| steps: | ||||||||
| - uses: actions/checkout@v5 | ||||||||
| - uses: actions/setup-python@v6 | ||||||||
| with: | ||||||||
| python-version: ${{ matrix.branch }} | ||||||||
| python-version: ${{ matrix.python-version }} | ||||||||
| allow-prereleases: true | ||||||||
| cache: pip | ||||||||
| - name: Clone docsbuild scripts | ||||||||
| Expand All | @@ -38,16 +41,16 @@ jobs: | |||||||
| --skip-cache-invalidation | ||||||||
| --theme "$(pwd)" | ||||||||
| --languages en | ||||||||
| --branches ${{ matrix.branch }} | ||||||||
| ${{ matrix.branch == '3.14' && '--select-output no-html' || '' }} | ||||||||
| --branches ${{ matrix.branch || matrix.python-version }} | ||||||||
| ${{ matrix.branch == 'main' && '--select-output no-html' || '' }} | ||||||||
|
Comment thread
Comment on lines
+44
to
+45
Copy link
Copy Markdown
Member
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. Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality
Suggested change
Sorry, something went wrong.
All reactions
|
||||||||
| - name: Show logs | ||||||||
| if: failure() | ||||||||
| run: | | ||||||||
| cat ./logs/docsbuild.log | ||||||||
| - name: Upload | ||||||||
| uses: actions/upload-artifact@v5 | ||||||||
| with: | ||||||||
| name: doc-html-${{ matrix.branch }} | ||||||||
| name: doc-html-${{ matrix.python-version }} | ||||||||
| path: www/ | ||||||||
|
|
||||||||
| translations: | ||||||||
| Expand Down | ||||||||
| Back | FazBrowse Home | New Git URL |
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.
Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low QualityA possible improvement is to compute the branch and extra opts once and assign them to env vars:
This keeps all the data and logic closer and avoids duplication. However we only need the branch and the extra opts once in the workflow, so your less-verbose solution is fine too.
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
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.
Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low QualityYeah, that is quite a bit more verbose, let's stick with the current version. Thanks!
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.