Skip to content

[Core] Move SceneCheckerVisitor out of the SceneChecking project into the core#6150

Open
bakpaul wants to merge 2 commits into
sofa-framework:masterfrom
bakpaul:26_06_enable_checking_in_python
Open

[Core] Move SceneCheckerVisitor out of the SceneChecking project into the core#6150
bakpaul wants to merge 2 commits into
sofa-framework:masterfrom
bakpaul:26_06_enable_checking_in_python

Conversation

@bakpaul

@bakpaul bakpaul commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

Moves the SceneCheck visitor into the core. This is required for the SofaPython3 PR enabling scene checking when launching a scene using python.

(Not a real depends-on, but so you see the effect)
[ci-depends-on https://github.com/sofa-framework/SofaPython3/pull/616]


By submitting this pull request, I acknowledge that
I have read, understand, and agree SOFA Developer Certificate of Origin (DCO).


Reviewers will merge this pull-request only if

  • it builds with SUCCESS for all platforms on the CI.
  • it does not generate new warnings.
  • it does not generate new unit test failures.
  • it does not generate new scene test failures.
  • it does not break API compatibility.
  • it is more than 1 week old (or has fast-merge label).

@bakpaul bakpaul added pr: fast merge Minor change that can be merged without waiting for the 7 review days pr: status to review To notify reviewers to review this pull-request labels Jun 11, 2026
@sofabot

sofabot commented Jun 11, 2026

Copy link
Copy Markdown
Collaborator

[ci-depends-on] detected during build #1.

To unlock the merge button, you must

@hugtalbot hugtalbot added this to the v26.06 milestone Jun 12, 2026
@sofabot

sofabot commented Jun 12, 2026

Copy link
Copy Markdown
Collaborator

[ci-depends-on] detected during build #2.

To unlock the merge button, you must

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr: fast merge Minor change that can be merged without waiting for the 7 review days pr: status to review To notify reviewers to review this pull-request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants