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

Doc: document generate-makefile.ci #19239

Merged
merged 1 commit into from
Feb 3, 2023

Conversation

bergzand
Copy link
Member

@bergzand bergzand commented Feb 3, 2023

Contribution description

This documents the generate-makefile.ci target

Testing procedure

Read the added section to the advanced build system tricks

Issues/PRs references

None

@bergzand bergzand added Area: doc Area: Documentation CI: skip compile test If set, CI server will run only non-compile jobs, but no compile jobs or their dependent jobs labels Feb 3, 2023
@bergzand bergzand requested a review from benpicco February 3, 2023 09:56
@benpicco benpicco added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Feb 3, 2023
@riot-ci
Copy link

riot-ci commented Feb 3, 2023

Murdock results

✔️ PASSED

fc471d6 Doc: document generate-makefile.ci

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

Artifacts

@maribu
Copy link
Member

maribu commented Feb 3, 2023

bors merge

@bors
Copy link
Contributor

bors bot commented Feb 3, 2023

Build succeeded:

@bors bors bot merged commit 9027179 into RIOT-OS:master Feb 3, 2023
@silkeh
Copy link
Contributor

silkeh commented Feb 3, 2023

Err... isn't the target create-Makefile.ci? 😕

bors bot added a commit that referenced this pull request Feb 4, 2023
19244: Make: Add generate-Makefile.ci r=maribu a=bergzand

### Contribution description

This is the same as the create-Makefile.ci. The alias is added to stay in line with other generate-* Make targets. Now the documentation should also be valid again

### Testing procedure

The target should now match the documentation in the doxygen.

### Issues/PRs references

follow-up to #19239 

Co-authored-by: Koen Zandberg <koen@bergzand.net>
@maribu
Copy link
Member

maribu commented Feb 5, 2023

The implementation now was updated to match the doc

@MrKevinWeiss MrKevinWeiss added this to the Release 2023.04 milestone Apr 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: doc Area: Documentation 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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants