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

@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

3 participants