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.

Sign up for GitHub

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

Jump to bottom

.murdock.yml: Add preview link to documentation to comment #18677

Merged
merged 1 commit into from
Sep 30, 2022
Merged

.murdock.yml: Add preview link to documentation to comment #18677

merged 1 commit into from
Sep 30, 2022

Conversation

miri64
Copy link
Member

@miri64 miri64 commented Sep 30, 2022

Contribution description

This will provide a link to the rendered documentation in Murdock's stick comment.

Testing procedure

Once murdock-ng/murdock#9 is deployed, we can set the "CI: ready for build" label. The comment by Murdock should then contain a link to the documentation preview.

Issues/PRs references

Requares murdock-ng/murdock#9 and the code in it being deployed.

@miri64 miri64 added Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation State: waiting for other PR State: The PR requires another PR to be merged first State: waiting for CI update State: The PR requires an Update to CI to be performed first labels Sep 30, 2022
@github-actions github-actions bot added the Area: CI Area: Continuous Integration of RIOT components label Sep 30, 2022
@miri64 miri64 added CI: skip compile test If set, CI server will run only non-compile jobs, but no compile jobs or their dependent jobs CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR and removed State: waiting for other PR State: The PR requires another PR to be merged first State: waiting for CI update State: The PR requires an Update to CI to be performed first labels Sep 30, 2022
@riot-ci
Copy link

riot-ci commented Sep 30, 2022

Murdock results

✔️ PASSED

f1c3be1 .murdock.yml: Add preview link to documentation to comment

Success Failures Total Runtime
1 0 1 01m:12s

Artifacts

This only reflects a subset of all builds from https://ci-prod.riot-os.org. Please refer to https://ci.riot-os.org for a complete build for now.

@miri64
Copy link
Member Author

miri64 commented Sep 30, 2022

Artifacts

* [Documentation preview](https://ci-prod.riot-os.org/results/43d8d157d4534265965861b1b7ae831c/doc-preview/)

It worked! :-)

aabadie
aabadie approved these changes Sep 30, 2022
View reviewed changes
Copy link
Contributor

@aabadie aabadie left a comment

Choose a reason for hiding this comment

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

ACK!

@aabadie aabadie enabled auto-merge September 30, 2022 14:17
@aabadie aabadie merged commit 366f7e1 into RIOT-OS:master Sep 30, 2022
@miri64 miri64 deleted the murdock_yml/enh/doc-preview-link branch September 30, 2022 15:44
@maribu maribu added this to the Release 2022.10 milestone Oct 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aabadie aabadie aabadie approved these changes

@kaspar030 kaspar030 Awaiting requested review from kaspar030

Assignees
No one assigned
Labels
Area: CI Area: Continuous Integration of RIOT components CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR CI: skip compile test If set, CI server will run only non-compile jobs, but no compile jobs or their dependent jobs Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation
Projects
None yet
Milestone
Release 2022.10
Development

Successfully merging this pull request may close these issues.
4 participants
@miri64 @riot-ci @aabadie @maribu