+
Skip to content

Conversation

MaximilianSoerenPollak
Copy link
Contributor

@MaximilianSoerenPollak MaximilianSoerenPollak commented Jun 5, 2025

Currently all needs that are external (like process) will be filtered out in the 'needs.json'.

This allows the needs.json to contain ALL of the needs imported.

In a future PR the filter can probably be adapted to make it more efficent, e.g. only get external needs that are actually needed.

## ⚠️ This PR should not be merged until new docs-as-code is out. As that has bugfixes that might be imported for here.

New docs-as-code version is available. PR can be reviewed and mered.


I pulled the version to a new major one, as the change in needs.json is something that changes how it could be used.
I think that makes sense (at least in my head) .

Copy link

github-actions bot commented Jun 5, 2025

The created documentation from the pull request is available at: docu-html

Increasing docs-as-code version
Increasing own module version
@MaximilianSoerenPollak MaximilianSoerenPollak changed the title Enable external needs json Enable external needs json & increase versions Jun 5, 2025
@MaximilianSoerenPollak MaximilianSoerenPollak marked this pull request as ready for review June 6, 2025 07:18
@MaximilianSoerenPollak MaximilianSoerenPollak marked this pull request as draft June 6, 2025 07:18
@MaximilianSoerenPollak MaximilianSoerenPollak marked this pull request as ready for review June 6, 2025 07:40
},
}
# This ensures all needs that are imported show up in the build 'needs.json'
needs_builder_filter = ""
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this belongs to wherever we have the external need stuff and not into individual conf.py files

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I put it into the conf.py here right now, as this can be done without a complete new release of docs-as-code.

In the next release we should add that in there as a default and figure out a better filter yes.

@MaximilianSoerenPollak MaximilianSoerenPollak merged commit 40dd375 into eclipse-score:main Jun 6, 2025
6 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

点击 这是indexloc提供的php浏览器服务,不要输入任何密码和下载