Skip to content

Conversation

@JoanMForgeFlow
Copy link
Contributor

Standard migration to version 15.0

@JoanMForgeFlow JoanMForgeFlow force-pushed the 15.0-mig-queue_job_cron branch from 0df6d93 to 103456e Compare March 1, 2022 08:30
Copy link

@AaronHForgeFlow AaronHForgeFlow left a comment

Choose a reason for hiding this comment

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

Functional review

Copy link

@mariadforgeflow mariadforgeflow left a comment

Choose a reason for hiding this comment

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

LGTM!

@OCA-git-bot
Copy link
Contributor

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). 🤖

Copy link
Contributor

@simahawk simahawk left a comment

Choose a reason for hiding this comment

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

LG

@simahawk
Copy link
Contributor

/ocabot merge nobump

@simahawk
Copy link
Contributor

/ocabot migration queue_job_cron

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 15.0-ocabot-merge-pr-410-by-simahawk-bump-nobump, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Mar 11, 2022
Signed-off-by simahawk
@OCA-git-bot OCA-git-bot added this to the 15.0 milestone Mar 11, 2022
@OCA-git-bot OCA-git-bot mentioned this pull request Mar 11, 2022
10 tasks
@OCA-git-bot
Copy link
Contributor

@simahawk your merge command was aborted due to failed check(s), which you can inspect on this commit of 15.0-ocabot-merge-pr-410-by-simahawk-bump-nobump.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

@AaronHForgeFlow
Copy link

I think the CI failed because of an infrastructure thing. Probably retrying the merge will succeed. Can you do so @simahawk ?

@simahawk
Copy link
Contributor

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 15.0-ocabot-merge-pr-410-by-simahawk-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit fbe33ad into OCA:15.0 Apr 29, 2022
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 4502fe5. 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