-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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.
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
[16.0][MIG] base_domain_inverse_function: Migrate to version 16.0 #2592
[16.0][MIG] base_domain_inverse_function: Migrate to version 16.0 #2592
Conversation
This PR has the |
@sonhd91 Thanks for the mig. I did update the original PR with pre-commit improvements and fixing the test. |
/ocabot merge nobump |
Hey, thanks for contributing! Proceeding to merge this for you. |
@gurneyalex your merge command was aborted due to failed check(s), which you can inspect on this commit of 16.0-ocabot-merge-pr-2592-by-gurneyalex-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. |
There seems to be an issue in the tests https://github.com/OCA/server-tools/actions/runs/4980917389/jobs/8914445178#step:8:307 I'm not sure if it is related to another module that was merged in the meantime... @sonhd91 can you rebase your PR? |
3dbfea8
to
250df30
Compare
reabase done, thanks |
b0ed8a8
to
95d62a3
Compare
95d62a3
to
7d69242
Compare
Good day @gurneyalex, the PR is now green for merging. Thanks in advance for your attention. |
/ocabot merge nobump |
On my way to merge this fine PR! |
Congratulations, your PR was merged at f4562f6. Thanks a lot for contributing to OCA. ❤️ |
Based upon: