🐛 feat(containers) enforce CGNS name validation across tree insertion

[casenave]

Checklist

• Typing enforced
• Documentation updated
• Changelog updated
• Tests and Example updates
• Coverage should be 100%

🔗 Related issues (optional)

Closes #312

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

[casenave]

Thanks a lot @daviddanan for trying the lib and proposing an issue :)
Do you think that the curreent PR solves your issue ?

[casenave]

Adding the check to the Muscat CGNSBridge as well, see this MR.

[daviddanan]

Thanks a lot @daviddanan for trying the lib and proposing an issue :) Do you think that the curreent PR solves your issue ?

You are welcome! I am still investigating to isolate some unexpected behaviour, expect other issues soon.
Yes, such error handling seems fine, thanks. It is clear enough to fix the issue on the user end.

edit: in fact, 32 characters is ok for CGNS so shorter or equal to 32 is acceptable.

[casenave]

edit: in fact, 32 characters is ok for CGNS so shorter or equal to 32 is acceptable.

Updated for this