Feat/generate ubproject and schema validation#394
Draft
arnoox wants to merge 255 commits intoeclipse-score:mainfrom
Draft
Feat/generate ubproject and schema validation#394arnoox wants to merge 255 commits intoeclipse-score:mainfrom
arnoox wants to merge 255 commits intoeclipse-score:mainfrom
Conversation
Ran update & updgrade on requirements.
* Renamed vars for readability * small improvements and cleanup * Moving of BUILD files * Moving string_links option * Formatting Preparation commit to move all py_libraries to their own BUILD files. This solution at the moment is WIP as the pre-fix needed is unacceptable. But this will serve as a base to be able to test if further changes work Moved BUILD files to separate folders for better Separation Cleaned up some smaller stuff Conf.py is cleaner again
Plucking 'docs/_tooling' from 'eclipse-score/score' to enable it as standalone 'Module'. * Adding commits from 'score' Some changes have been made by 'mmr1909'. The original/referenced commit is this one: eclipse-score/score@8e8646b Adding commits from 'score' regarding the 'docs/_tooling' folder as is. The original commits were by: hoe-jo The commits can be found here: eclipse-score/score@233c03d eclipse-score/score@5d1200e
* Prefix name with score * Name change to comply with standards
…score#19) by: Aymen Soussi aymen.soussi@expleogroup.com
* Fixed source_code_linker finding external needs * Added explaining comments Added simple loop search logic to try all available prefixes per id.
Include inc files in doc build
Signed-off-by: Maximilian Sören Pollak <maximilian.pollak@expleogroup.com>
- Added target for building Github archive - Take the input files an put them in the archive as-is. No renaming. - Use empty lists for now, as we are still not sure how to use or adapt the source code linker in multirepo Addresses: eclipse-score#16 --------- Signed-off-by: Dan Calavrezo <dan.calavrezo.ext@qorix.ai>
updated version for release Addresses: eclipse-score#16 Signed-off-by: Dan Calavrezo <dan.calavrezo.ext@qorix.ai>
* Fixed wrong check activation * Fixed test & rst files Test and rst files needed fixing to comply with new check rules Check found False positives due to 'process' being moved and loosing it's prefix.
Cleaned up unused imports and small fix inside the README Also-by: Aymen Soussi aymen.soussi@expleogroup.com
- Add multiple 'uni-direcional' docs build examples - Bugfix 'docs_needs' build target in Module imports - Patch some spelling mistakes - Delete 'docs'. It has been moved to examples/simple already. --- Addresses a bug that caused the 'docs_needs' import via a Module (like in the linking-release example) to not be executed correctly, as it would be missing the dependencies and missing the 'sphinx_build binary'
Signed-off-by: Alexander Lanin <Alexander.Lanin@etas.com>
* Add bugfix for new module json_encoding quirks
Remove defined external 'id_prefixes' from to be checked links. Added another example to have an rst file inside a folder as well. Added some explanation to descriptions of needs
Replaced the markdown table with a reStructuredText table format in the description of reqid: Safety Analysis Linkage Violates" Signed-off-by: Chidananda Swamy R <chidananda.swamy@ltts.com>
arnoox
force-pushed
the
feat/generate-sn6-schema-validation
branch
from
71d21c1 to
29fcd07
Compare
|
Thanks for the pull request. This thread has been quiet for 30 days, so we are marking it as stale for now. Please take a quick look and let us know whether it is still up to date, still relevant, needs review, or is ready to merge. Any new activity will remove the stale label automatically. If nothing changes in the next 10 days, we will close it to keep the backlog current. |
added 8 commits
April 13, 2026 09:45
Resolved conflicts in 4 files: - score_sync_toml/__init__.py: keep needscfg_exclude_vars (PR) + use .append() (upstream) - score_metamodel/__init__.py: use upstream SN6 API (needs_links/needs_fields), remove Any import - score_source_code_linker/__init__.py: use upstream module_grouped_needs.repo + 2-arg get_github_link - requirements.rst: keep needextend for metamodel.yaml source_code_link (PR feature)
…com/arnoox/score-docs-as-code into feat/generate-sn6-schema-validation
arnoox
changed the title
added 5 commits
April 13, 2026 15:13
When merging global_base_opts into optional_options in _parse_need_type(),
fields that are already declared as mandatory_options for that type were
being duplicated. This caused:
- Schema generation to emit a conflicting pattern ('^$|^.*$' instead of
'^.*$') when the same field appeared in both optional and mandatory.
- test_optional_options_not_required: 'version' for doc_tool was listed
in optional_options despite being mandatory, causing a test failure.
Fixes:
- yaml_parser.py: filter global_base_opts before merging into optional_options
- sn_schemas.py: defensive guard to skip optional fields already in mandatory
- TestCompSchema._make_valid(): add 'belongs_to' mandatory link (comp requires belongs_to: feat) - TestFeatSchema: feat.includes is an optional link, not mandatory; replace failing link tests with tests for mandatory options (security, safety) that are actually required by the schema
The needextend directive was adding ':+satisfies: tool_req__docs_metamodel' to all needs with 'metamodel.yaml' in their source_code_link. However, the need 'tool_req__docs_metamodel' is not defined anywhere, causing 39 '[needs.link_outgoing]' Sphinx warnings which are fatal under -W.
…mpfile for merging
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
20 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
📌 Description
🚨 Impact Analysis
✅ Checklist