Archives
Contributors
Re: How to use /ocabot merge correctly?
About the version number idea, it's depicted in contributing guidelines:
Regards.
by Pedro M. Baeza - 04:46 - 17 Feb 2020
Reference
-
How to use /ocabot merge correctly?
Dear community,
We have seen this document, yet it not tell in great detail. https://odoo-community.org/blog/the-oca-blog-1/post/the-oca-github-bot-can-now-merge-pull-requests-92 so may we ask in detailed here,1) What is the difference between /ocabot merge patch/minor/major and the symantic of x.y.z - breaking.feature.fix in another document. Is it so that,- /ocabot merge patch, 13.0.1.0.0 -> 13.0.1.0.1 (patch = fix)- /ocabot merge minor, 13.0.1.0.0 -> 13.0.1.1.0 (minor = feature)- /ocabot merge major, 13.0.1.0.0 -> 13.0.2.0.0 (major = breaking)2) For the new [ADD] module, should be click merge manually, or use /ocabot merge (don't want to add any digit, but we test it and it seem to try to change last digit).3) What is "/ocabot merge" alone, it default to patch or minor?4) What is the practice to work with some module that has HISTORY. For example, some author (not the PSC), may add in history as new Minor revision, 13.0.1.0.0 -> 13.0.1.1.0 but the merger think it is just a fix. And so, he might do /ocabot merge patch . This could result in the version diff between HISTORY and manifest file?Thank you!Kitti U.
by Kitti Upariphutthiphong - 10:10 - 17 Feb 2020-
Re: How to use /ocabot merge correctly?
Thank you for the explanation. I am more clear now.Can I help modify this blog post? https://odoo-community.org/blog/the-oca-blog-1/post/the-oca-github-bot-can-now-merge-pull-requests-92Note: it seem everyone could edit the page, correct?On Mon, Feb 17, 2020 at 10:47 PM Pedro M. Baeza (Tecnativa) <pedro.baeza@tecnativa.com> wrote:About the version number idea, it's depicted in contributing guidelines:Regards.
_______________________________________________
Mailing-List: https://odoo-community.org/groups/contributors-15
Post to: mailto:contributors@odoo-community.org
Unsubscribe: https://odoo-community.org/groups?unsubscribe
by Kitti Upariphutthiphong - 08:46 - 18 Feb 2020 -
Re: How to use /ocabot merge correctly?
About the version number idea, it's depicted in contributing guidelines:Regards.
by Pedro M. Baeza - 04:46 - 17 Feb 2020 -
Re: How to use /ocabot merge correctly?
Hi Kitti,Thanks for raising this topic. Here are some answer elements.1/ I'm not sure if there is a document explaining it but the idea is to roughly use it in semver style so:- patch: for code-only bug fixes that do not require -u to install- minor: when a database update (-u) is needed (for view changes, new fields) but the code remains otherwise compatible (no existing method signature change, no removed fields...).- major: when existing method signature change, or fields or models are removed or changed2/ for new modules or migration, always use /ocabot merge too, because it will run all tests again before merging. I personally tend to use /ocabot merge patch for new modules too, because it has the added benefit to publish a wheel with a round number.3/ /ocabot merge alone does not bump version. It should be used rarely, There is this idea of deprecating the bare "/ocabot merge" in favor of "/ocabot merge nobump".4/ regarding HISTORY.rst (the changelog), I'm proposing to use towncrier (see this issue for more information) to avoid conflicts and generate it from fragments.All this is of course open for improvement :)Best regards,-sbiOn Mon, Feb 17, 2020 at 10:11 AM Kitti Upariphutthiphong <kittiu@ecosoft.co.th> wrote:Dear community,
We have seen this document, yet it not tell in great detail. https://odoo-community.org/blog/the-oca-blog-1/post/the-oca-github-bot-can-now-merge-pull-requests-92 so may we ask in detailed here,1) What is the difference between /ocabot merge patch/minor/major and the symantic of x.y.z - breaking.feature.fix in another document. Is it so that,- /ocabot merge patch, 13.0.1.0.0 -> 13.0.1.0.1 (patch = fix)- /ocabot merge minor, 13.0.1.0.0 -> 13.0.1.1.0 (minor = feature)- /ocabot merge major, 13.0.1.0.0 -> 13.0.2.0.0 (major = breaking)2) For the new [ADD] module, should be click merge manually, or use /ocabot merge (don't want to add any digit, but we test it and it seem to try to change last digit).3) What is "/ocabot merge" alone, it default to patch or minor?4) What is the practice to work with some module that has HISTORY. For example, some author (not the PSC), may add in history as new Minor revision, 13.0.1.0.0 -> 13.0.1.1.0 but the merger think it is just a fix. And so, he might do /ocabot merge patch . This could result in the version diff between HISTORY and manifest file?Thank you!Kitti U._______________________________________________
Mailing-List: https://odoo-community.org/groups/contributors-15
Post to: mailto:contributors@odoo-community.org
Unsubscribe: https://odoo-community.org/groups?unsubscribe
by Stéphane Bidoul - 04:40 - 17 Feb 2020
-