Closed baolanlequang closed 4 months ago
397dac3
during Continuous Integration #2936Summary coverage rate: lines......: 64.2% (13676 of 21316 lines) functions..: no data found branches...: no data found Files changed coverage rate: n/a
af49da5
during Continuous Integration #2972Summary coverage rate: lines......: 64.1% (13673 of 21317 lines) functions..: no data found branches...: no data found Files changed coverage rate: n/a
a943f26
during Continuous Integration #3042Summary coverage rate: lines......: 64.2% (13683 of 21319 lines) functions..: no data found branches...: no data found Files changed coverage rate: n/a
[x] rather 1-story 1-commit than sub-atomic commits
[x] commit title is meaningful => git history search
[ ] commit description is helpful => helps the reviewer to understand the changes
[ ] code is up-to-date with the latest developments of the target branch (rebased to it or whatever) => :fast_forward:-merge for linear history is favoured
[ ] added code is linted
[ ] tests are passing (at least locally): we still have some random test failure on CI. thinking of asking spec/examples.txt to be commited
[x] in case the changes are visible to the end-user, video or screenshots should be added to the PR => helps with user testing
https://github.com/ComPlat/chemotion_ELN/assets/9006482/3b2f4051-7e42-42c7-ad5d-5c9f9d9cb53d
[ ] testing coverage improvement is improved.
[ ] CHANGELOG : add a bullet point on top (optional: reference to github issue/PR )
[ ] parallele PR for documentation on docusaurus if the feature/fix is tagged for a release