8000 Use version_info instead of parse_version by sbidoul · Pull Request #124 · acsone/click-odoo-contrib · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Use version_info instead of parse_version #124

8000
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

Merged
merged 2 commits into from
Oct 18, 2022
Merged

Use version_info instead of parse_version #124

merged 2 commits into from
Oct 18, 2022

Conversation

sbidoul
Copy link
Member
@sbidoul sbidoul commented Oct 18, 2022

This is a more robust pattern.

@codecov-commenter
Copy link
codecov-commenter commented Oct 18, 2022

Codecov Report

Merging #124 (bdf53dc) into master (f3298b0) will decrease coverage by 0.40%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##           master     #124      +/-   ##
==========================================
- Coverage   92.27%   91.86%   -0.41%     
==========================================
  Files          13       13              
  Lines         880      873       -7     
  Branches      154      152       -2     
==========================================
- Hits          812      802      -10     
- Misses         46       50       +4     
+ Partials       22       21       -1     
Impacted Files Coverage Δ
click_odoo_contrib/initdb.py 95.81% <50.00%> (-0.09%) ⬇️
click_odoo_contrib/update.py 85.45% <100.00%> (+0.93%) ⬆️
click_odoo_contrib/dropdb.py 75.00% <0.00%> (-20.84%) ⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@sbidoul sbidoul merged commit 1c15e02 into master Oct 18, 2022
@sbidoul sbidoul deleted the version_info branch October 18, 2022 15:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants
0