Skip to content

Conversation

PhilipPartsch
Copy link
Contributor

implement #1238

@PhilipPartsch
Copy link
Contributor Author

@chrisjsewell could you please check the failing test? I am not sure to overcome this.
I think the rest is fine, could you review the rest?

@chrisjsewell
Copy link
Member

I think the rest is fine, could you review the rest?

Heya, I'm just working on the internal representation of needs (for scaling), and how it is exposed, so I would hold off on reviewing/merging this yet.
I understand the use-case though

@chrisjsewell
Copy link
Member

Heya, note you can now use https://sphinx-needs.readthedocs.io/en/latest/api.html#sphinx_needs.api.need.get_needs_view, to retrieve all needs

@PhilipPartsch
Copy link
Contributor Author

@chrisjsewell can we merge it now?

@ubmarco
Copy link
Member

ubmarco commented Jul 24, 2025

Instead of expanding on this PR I vote to move over to #1467 once that's merged. It supports the separation of local and network checks and should tick all boxes for your use cases.
It also comes with a schema builder that just checks for compliance.

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.

3 participants