Skip to content

[DOC] add example to merge command#346

Merged
OCA-git-bot merged 1 commit intoOCA:masterfrom
madmooose:explain_merge_attributes
Apr 23, 2026
Merged

[DOC] add example to merge command#346
OCA-git-bot merged 1 commit intoOCA:masterfrom
madmooose:explain_merge_attributes

Conversation

@madmooose
Copy link
Copy Markdown
Contributor

No description provided.

@madmooose madmooose force-pushed the explain_merge_attributes branch from c93b144 to 3ce2906 Compare April 23, 2026 14:46
Comment thread README.rst
* merge the PR onto a temporary branch created off the target branch
* merge when tests on the rebased branch are green
* optionally bump the version number of the addons modified by the PR
* optionally bump the version number of the addons modified by the PR. (e.g. 19.0. ``major`` . ``minor`` . ``patch``)
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why 19.0 ?

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As an example only. What would you think?

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you explain the example? I don't quite understand it.

Copy link
Copy Markdown
Contributor Author

@madmooose madmooose Apr 23, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ValentinVinagre
Long version of the example for 19.1.0.0

  • major: 19.0.2.0.0
  • minor: 19.0.1.1.0
  • patch: 19.0.1.0.1
  • nobump: 19.0.1.0.0

I'm open for a more understandable version to make this clear to everybody

@OSevangelist
Copy link
Copy Markdown

lgtm

@OSevangelist
Copy link
Copy Markdown

/ocabot merge nobump

@OCA-git-bot
Copy link
Copy Markdown
Contributor

This PR looks fantastic, let's merge it!
Prepared branch master-ocabot-merge-pr-346-by-OSevangelist-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit eb48c96 into OCA:master Apr 23, 2026
2 checks passed
@OCA-git-bot
Copy link
Copy Markdown
Contributor

Congratulations, your PR was merged at eb48c96. Thanks a lot for contributing to OCA. ❤️

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.

5 participants