OCA
[16.0][IMP] subscription_oca: Various improvements
Fixes improvements mentioned here: https://github.com/OCA/contract/issues/1057
Not sure why the codecov/patch check is failing because codecov says there's a positive change (+0.03%)
There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.
![github-actions[bot] avatar](https://avatars.githubusercontent.com/in/15368?v=4 "Published by a pub.dev verified publisher"){kind=small}
There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.
@OCA/community-maintainers There's no team for contract maintainers. Can anyone decide if this can be merged or not, please. Codecov is failing but I don't know how I can increase test coverage in a meaningful way without creating bs tests.
@rousseldenis Thanks! I will look this week on how to extend the tests (in a meaningful way) to cover more code.
This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖
@rousseldenis @OCA/project-service-maintainers Can we please merge this? Let me know
What a great day to merge this nice PR. Let's do it! Prepared branch 16.0-ocabot-merge-pr-1058-by-dreispt-bump-minor, awaiting test results.
Congratulations, your PR was merged at 158cbe499dae6c652ec500657e3d45d403d292fb. Thanks a lot for contributing to OCA. ❤️