Skip to content
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

Creating multichip docs with GitHub Pages #1243

Closed
wants to merge 2 commits into from

Conversation

playfulFence
Copy link
Contributor

IMPORTANT: It requires to also create a branch were the web-page will be deployed named "gh-pages"

ci.yml has new job, which utilizes new build-docs.sh simple shell script. Basically it uses nightly and xtensa(/esp) toolchains to build documentation using xtask (special thanks to @jessebraham for the latest update of xtask).

The web-page is static and located in resources/index.html (in future we're able to use "plain" (since we're not using server) JavaScript for some smaller dynamic parts, if needed)

PS: progress made for #1204

@playfulFence playfulFence self-assigned this Mar 5, 2024
@igrr
Copy link

igrr commented Mar 5, 2024

Just a suggestion, instead of using the gh-pages branch, you can now deploy to GitHub Pages directly. See the official action: https://github.com/actions/deploy-pages and an example: https://github.com/igrr/edc23/blob/885ded3bbb68034a2a7acc42e67c1aa3de05dd25/.github/workflows/deploy.yml#L36.

(You also need to enable the option in the repo settings: https://docs.github.com/en/pages/getting-started-with-github-pages/configuring-a-publishing-source-for-your-github-pages-site#publishing-with-a-custom-github-actions-workflow)

@jessebraham jessebraham marked this pull request as draft March 6, 2024 13:54
CHIPS=("esp32" "esp32c2" "esp32c3" "esp32c6" "esp32h2" "esp32p4" "esp32s2" "esp32s3")
for CHIP in "${CHIPS[@]}"; do
if [ "$CHIP" == "esp32" ] || [ "$CHIP" == "esp32s2" ] || [ "$CHIP" == "esp32s3" ]; then
cargo +esp xtask build-documentation --output-path=docs/"$CHIP"/ esp-hal "$CHIP"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+esp has the riscv targets too, so can't we always use the esp toolchain?

@@ -296,3 +296,28 @@ jobs:
run: cargo fmt --all --manifest-path=esp-riscv-rt/Cargo.toml -- --check
- name: rustfmt (examples)
run: cargo fmt --all --manifest-path=examples/Cargo.toml -- --check

build-docs:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should not be running for every commit pushed to every PR. We should either create a proper release workflow, or just create a new documentation workflow which is triggered by the workflow_dispatch event for now, so we can manually invoke it.

@@ -28,6 +28,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
- Ensuring that the random number generator is TRNG. (#1200)
- ESP32-C6: Add timer wakeup source for deepsleep (#1201)
- Introduce `InterruptExecutor::spawner()` (#1211)
- Creating multichip docs with GitHub Pages (#1243)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Documentation does not affect the user-facing API in any way, there is no need for a CHANGELOG.md entry in this PR.

@MabezDev MabezDev added the skip-changelog No changelog modification needed label Mar 6, 2024
@jessebraham jessebraham mentioned this pull request Mar 6, 2024
4 tasks
@jessebraham
Copy link
Member

jessebraham commented Mar 6, 2024

@playfulFence is on business travel, I will take over this work.

@jessebraham
Copy link
Member

This has been superseded by #1248, thanks again for your work on this!

@jessebraham jessebraham closed this Mar 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
skip-changelog No changelog modification needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants