test: add docs() tests#665
Conversation
License Check Results🚀 The license check job ran with the Bazel command: bazel run --lockfile_mode=error //src:license-checkStatus: Click to expand output |
MaximilianSoerenPollak
left a comment
There was a problem hiding this comment.
Looks quiet close, I had some questions but overall great solid start.
I think we can soon merge this
| # FIXME: | ||
| # sphinx-needs refuses to add an external need whose type is not registered in | ||
| # this project's needs_types, so the consumer must declare the producer's | ||
| # `test_req` type. score_metamodel's own checks skip external needs, so the | ||
| # id/parts rules never run here. Loaded via the score_metamodel_yaml config | ||
| # value because docs(metamodel=...) does not reach the build target. |
There was a problem hiding this comment.
You could just add the test_req to the config here no?
That would make for a simpler / nicer way of fixing this issue?
well okay not that simple, if I look at how it is done in the rst tests:
Though that is more complex due to it defining NEW options that all need to have. Which you don't need, so yo ucan make it simpl.er
There was a problem hiding this comment.
this is just temp, we need to fix the underlying issue in docs(). Then this can be removed completely.
MaximilianSoerenPollak
left a comment
There was a problem hiding this comment.
Seems fine for me now.
All questions answered, good start with a framework to continue building on top of it.
📌 Description
🚨 Impact Analysis
✅ Checklist