Contributors mailing list archives
contributors@odoo-community.org
Browse archives
Re: Changes in Ocabot - nobump will be now required
Re: Changes in Ocabot - nobump will be now required
Re: Changes in Ocabot - nobump will be now required
by
Acsone SA/NV, Stéphane Bidoul
Hi Denis,
For migration, where you already have the version number in the manifest you have two options
- /ocabot merge nobump: will keep the version in the manifest, the drawback is that you don't have a wheel with a round version number (it will get a .dev suffix depending on the number of commits in the module since the commit that set the manifest version)
- /ocabot merge patch: will do a patch version and a round wheel version number, the drawback being that you don't have a 1.0.0 and you go right away to 1.0.1
Tradeoffs... :)
Best regards,
-sbi
On Tue, Apr 14, 2020 at 10:52 AM Roussel, Denis <denis.roussel@acsone.eu> wrote:
Thanks for the great job !One question: for migration PR's, what's the best practice (as the good version number is already set) ?On Tue, Apr 14, 2020 at 10:36 AM Sylvain LE GAL <sylvain.legal@grap.coop> wrote:Hi all,We recently worked with Stephane Bidoul from Acsone, to improve the OCA bot that makes a lot of automated tasks on github. (https://github.com/OCA/oca-github-bot/)FIY, the merge command syntax will change (PR). Previously, it was possible to say \ocabot merge. It will be now mandatory to mention the bump method to apply, among these 4 options : major, minor, patch, or nobump.The nobump option should be used in rare cases. Most of the time, please follow the OCA conventions, regarding version numbers, available here.Remark : if the command is incorrect, the bot will now answer that the command was wrong. Sample.Another changes are available for the OCA bot :* Possibility to automatically update history.rst file, using oca-towncrier and fragments files (PR). If you're interested in such a feature, please take a look at that new OCA command, available in maintainer-tools project.Note 1 : There are also pending works to have the possibility to rebase the current pull request (PR) or to (auto) squash the commit of a PR. Feel free to contribute !Note 2 : For people that have installed a custom ocabot for their custom repositories, new settings are available.* Possibility to have approvals required configurable. (PR)* Possibility to add options, calling oca-gen-addons-table, oca-gen-addon-readme, oca-gen-addon-icon (PR)Kind regards._______________________________________________
Mailing-List: https://odoo-community.org/groups/contributors-15
Post to: mailto:contributors@odoo-community.org
Unsubscribe: https://odoo-community.org/groups?unsubscribe
--__________________________________________
Denis Roussel
Software Engineer
Acsone SA, Succursale de Liège (Val Benoît)
Tel : +32 2 888 31 49
Fax : +32 2 888 31 59
Gsm : +32 472 22 34 64Acsone sa/nv
Boulevard de la Woluwe 56 Woluwedal | B-1200 Brussels | BelgiumQuai Banning, 6 (Val Benoît) | B-4000 Liège | Belgium
Zone Industrielle 22 | L-8287 Kehlen | Luxembourg_______________________________________________
Mailing-List: https://odoo-community.org/groups/contributors-15
Post to: mailto:contributors@odoo-community.org
Unsubscribe: https://odoo-community.org/groups?unsubscribe
Reference
-
Changes in Ocabot - nobump will be now required
byGroupement Régional Alimentaire de Proximité, Sylvain LE GAL-
Re: Changes in Ocabot - nobump will be now required
byTAKOBI s.r.l., Lorenzo Battistini. -
Re: Changes in Ocabot - nobump will be now required
byAcsone SA/NV, Stéphane Bidoul -
Re: Changes in Ocabot - nobump will be now required
byAGILE BUSINESS GROUP ITALIA SRL, Alex Comba. -
Re: Changes in Ocabot - nobump will be now required
byCamptocamp SA, Simone Orsi -
Re: Changes in Ocabot - nobump will be now required
byClosingAp Open Source Integrators Europe, LDA, Daniel Reis -
Re: Changes in Ocabot - nobump will be now required
byAcsone SA/NV, Stéphane Bidoul -
Re: Changes in Ocabot - nobump will be now required
byOpen Source Integrators, Maxime Chambreuil -
Re: Changes in Ocabot - nobump will be now required
byAcsone SA/NV, Laurent Mignon
-