-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
[MIG] cron_inactivity_period: Migration to 12.0 #2455
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
[MIG] cron_inactivity_period: Migration to 12.0 #2455
Conversation
4948d86 to
fc03c12
Compare
legalsylvain
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for porting this module
Note : empty files in readme folder are not nexessary.
2db13a4 to
205b4ac
Compare
|
/ocabot migration cron_inactivity_period /ocabot merge nobump |
|
On my way to merge this fine PR! |
|
@thomaspaulb your merge command was aborted due to failed check(s), which you can inspect on this commit of 12.0-ocabot-merge-pr-2455-by-thomaspaulb-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. |
|
@ioans73 This needs:
|
|
@thomaspaulb your merge command was aborted due to failed check(s), which you can inspect on this commit of 12.0-ocabot-merge-pr-2455-by-thomaspaulb-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. |
|
This PR has the |
|
This PR has the |
205b4ac to
67bca70
Compare
67bca70 to
abbedfe
Compare
@thomaspaulb done! |
|
/ocabot migration cron_inactivity_period /ocabot merge nobump |
|
Hey, thanks for contributing! Proceeding to merge this for you. |
|
Congratulations, your PR was merged at fe6f8cb. Thanks a lot for contributing to OCA. ❤️ |
Standard migration