change to latest main for new sphinx needs linkage#82
change to latest main for new sphinx needs linkage#82RolandJentschETAS wants to merge 2 commits into
Conversation
License Check Results🚀 The license check job ran with the Bazel command: bazel run --lockfile_mode=error //:license-checkStatus: Click to expand output |
|
The created documentation from the pull request is available at: docu-html |
rmaddikery
left a comment
There was a problem hiding this comment.
Please address the comments
|
Why are QNX8 Builds skipped on the PR? |
No idea, but I have heard rumors that some QNX builds have to be manually approved and started. The background are some security risks for the pipelines (It is open source, therefore anyone can raise an PR and the PR could change the workflow in a way, that the QNX license which is used by the workflow is compromised or forwarded to the attacker). This approvement is only possible by committers, which rules out that. |
|
@copilot resolve the merge conflicts in this pull request |
Merge conflicts resolved in commit 62b6741. The conflicts in |
63cd7a4 to
2404cc1
Compare
2404cc1 to
676b5e4
Compare
Done. |
| "@score_process//:needs_json", | ||
| ], | ||
| source_dir = "docs", | ||
| source_dir = ".", |
There was a problem hiding this comment.
this is not yet supported by docs-as-code. bazel build //:needs_json will fail. Therefore logging cannot be a dependency to any other module.
Notes for Reviewer
Change to latest sphinx linking
Pre-Review Checklist for the PR Author
Checklist for the PR Reviewer
Post-review Checklist for the PR Author
References
Closes #