Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Add Copier #958

Merged
merged 2 commits into from
May 16, 2023
Merged

Add Copier #958

merged 2 commits into from
May 16, 2023

Conversation

SirTakobi
Copy link

@OCA-git-bot
Copy link
Contributor

Hi @Tardo,
some modules you are maintaining are being modified, check this out!

@SirTakobi SirTakobi mentioned this pull request Feb 27, 2023
Closed
@SirTakobi SirTakobi marked this pull request as ready for review February 27, 2023 13:34
@SirTakobi SirTakobi mentioned this pull request Feb 27, 2023
Closed
@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). 🤖

@dreispt
Copy link
Member

dreispt commented May 10, 2023

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 12.0-ocabot-merge-pr-958-by-dreispt-bump-nobump, awaiting test results.

OCA-git-bot added a commit that referenced this pull request May 10, 2023
@OCA-git-bot
Merge PR #958 into 12.0
87ab1b2 
Signed-off-by dreispt
@OCA-git-bot
Copy link
Contributor

@dreispt your merge command was aborted due to failed check(s), which you can inspect on this commit of 12.0-ocabot-merge-pr-958-by-dreispt-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.

SirTakobi added 2 commits May 16, 2023 10:18
@SirTakobi
Hello world 🖖
7add373
@SirTakobi
[REF] website_blog_excerpt_img: License not allowed
60e80e7 
Dependency html_image_url_extractor is AGPL and LGPL Modules cannot depend on AGPL Modules
@SirTakobi
Copy link
Author

Thanks @dreispt for having a look, unfortunately the CI failed due to odoo/odoo#114930.
I have updated the configuration to include OCA/oca-addons-repo-template#189 that should fix the issue.
Can you try again?

@dreispt
Copy link
Member

dreispt commented May 16, 2023

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 12.0-ocabot-merge-pr-958-by-dreispt-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 487aad2 into OCA:12.0 May 16, 2023
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 9ef77ae. 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
Reviewers

@mathben mathben mathben approved these changes

@matteoopenf matteoopenf matteoopenf approved these changes

Assignees
No one assigned
Labels
approved merged 🎉 ready to merge
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@SirTakobi @OCA-git-bot @dreispt @mathben @matteoopenf