New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Changelog] Update Changelog with breaking changes #1562
[Changelog] Update Changelog with breaking changes #1562
Conversation
… with recent breaking PRs
- [All] Uniform size type [#1515](https://github.com/sofa-framework/sofa/pull/1515) | ||
- [All] Change index_type from size_t to uint [#1514](https://github.com/sofa-framework/sofa/pull/1514) | ||
- [All] Standardize index type for Vector/Matrix templates [#1453](https://github.com/sofa-framework/sofa/pull/1453) | ||
- Ongoing [cleaning of the Link implementation](https://github.com/sofa-framework/sofa/issues/1526) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would break the breaking into 2 parts here:
- The big big breaking (sofa ng + size_t) and the usual breaking
|
||
- Ongoing [SOFA-NG project](https://github.com/sofa-framework/sofa/issues/1527) generate several breaking changes due to modularization. It currently targets SOFA packages (SofaMisc, [SofaGeneral](https://github.com/sofa-framework/sofa/issues/1529) and soon SofaCommon), as well as SofaKernel (SofaBase\*, SofaCore, SofaHelper, SofaSimulation). Follow the [progress of the project online](https://github.com/sofa-framework/sofa/projects/9#card-47075575). | ||
- [Modularization] SofaMisc* (Revival) [#1520](https://github.com/sofa-framework/sofa/pull/1520) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would put a line before like:
Any cmake configuration error could come from PR Y
and before the size_t change: any compilation error link to type missmatch or unknow signature method could come from PR X
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks for the feedback @epernod
Update ChangeLog to better guide users when facing issues with recent breaking PRs
This PR:
Reviewers will merge only if all these checks are true.