Skip to content

add .github/workflows/build-test.yml for development.#119

Merged
kydos merged 1 commit intozenoh-rs:masterfrom
fujitatomoya:enable-github-workflow
Feb 2, 2026
Merged

add .github/workflows/build-test.yml for development.#119
kydos merged 1 commit intozenoh-rs:masterfrom
fujitatomoya:enable-github-workflow

Conversation

@fujitatomoya
Copy link
Contributor

This is enhancement for CI/CD to verifies that the PR fix does not break the documentation.
This github workflow checks every commit (or push to the development branch) to make sure hugo can build the documentation without any errors.

Signed-off-by: Tomoya Fujita <Tomoya.Fujita@sony.com>
Copy link
Contributor Author

@fujitatomoya fujitatomoya left a comment

Choose a reason for hiding this comment

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

I think this kind of workflow for documentation CI/CD pipeline is necessary to keep the quality and maintainability to make sure each PR does not break the documentation.

@JEnoch same here. no rush, when you have time, please have a look 😄

build:
runs-on: ubuntu-22.04
steps:
- uses: actions/checkout@v2
Copy link
Contributor Author

Choose a reason for hiding this comment

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

this is kinda old... but i just aligned with other workflow file. we probably could enable dependabot to upgrade those actions in the future.

Comment on lines +30 to +40
- name: Verify build output
run: |
if [ ! -d "public" ]; then
echo "Error: public directory not created"
exit 1
fi
if [ ! -f "public/index.html" ]; then
echo "Error: index.html not found in public directory"
exit 1
fi
echo "Build successful!"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

if hugo could not build the documentation successfully, this detects the failure. and then CI failure tells the developer needs to address the failure via workflow.

@fujitatomoya
Copy link
Contributor Author

@sashacmc @JEnoch friendly ping!

@kydos
Copy link
Contributor

kydos commented Feb 2, 2026

@fujitatomoya thanks for the contribution. Merging it!

@kydos kydos merged commit 2e263c3 into zenoh-rs:master Feb 2, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants