change to latest main for new sphinx needs linkage#82
change to latest main for new sphinx needs linkage#82RolandJentschETAS wants to merge 27 commits intomainfrom
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 |
Co-authored-by: Raghavendra Maddikery <raghav.maddikery@gmail.com> Signed-off-by: RolandJentschETAS <135332348+RolandJentschETAS@users.noreply.github.com>
Co-authored-by: Raghavendra Maddikery <raghav.maddikery@gmail.com> Signed-off-by: RolandJentschETAS <135332348+RolandJentschETAS@users.noreply.github.com>
rmaddikery
left a comment
There was a problem hiding this comment.
Please address the comments
| ) | ||
|
|
||
| bazel_dep(name = "score_tooling", version = "1.2.0") | ||
| git_override( |
There was a problem hiding this comment.
AFAIU we should no longer use git_override as per bzlmod guidelines. Any particular reason we still need it?
There was a problem hiding this comment.
Yes. Tooling have not provided a version for this. And the bazel rules in S-CORE says that is normal until the next version. I hope always the tooling and docs_as_code make versions in case of major bugfixes. Unfortunately that's not always the case.
There was a problem hiding this comment.
I have asked tooling if and when they provide a new release ...
There was a problem hiding this comment.
Tooling is in rework ... Not sure, when they will provide a new version.
Co-authored-by: Raghavendra Maddikery <raghav.maddikery@gmail.com> Signed-off-by: RolandJentschETAS <135332348+RolandJentschETAS@users.noreply.github.com>
|
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 |
Resolve conflicts in MODULE.bazel and MODULE.bazel.lock: - Use released versions of score_tooling (1.1.2), score_process (1.5.4), score_platform (0.5.5) from main instead of git_overrides - Use score_bazel_platforms 0.1.2 from main - Update MODULE.bazel.lock to match resolved dependencies Co-authored-by: RolandJentschETAS <135332348+RolandJentschETAS@users.noreply.github.com>
Merge conflicts resolved in commit 62b6741. The conflicts in |
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 #