Repository navigation
Conversation
0b87929 to
4e12088
Compare
|
Hi @sbidoul, I rebased this PR on the latest 19.0, removed the temporary test-requirements.txt pointing to refs/pull/840/head, restored requirements.txt, and squashed everything into a single commit. CI is green on the new head. |
|
Please do different PRs for queue_job_cron and queue_job_cron_jobrunner. Despite the similar name they are very different things. |
Split from OCA#843 as requested; this commit only migrates queue_job_cron.
4e12088 to
092f849
Compare
Split from OCA#843 as requested; this commit only migrates queue_job_cron_jobrunner.
Split from OCA#843 as requested; this commit only migrates queue_job_cron_jobrunner.
|
For what it’s worth: we’ve been using this module version in production for some time, and it works fine. |
|
We've also been using this module in production for a long time now. It works very well. |
|
This PR has the |
|
Any idea on when this will be merged ? since now there is an 18 version in the 19 report |
|
ping @sbidoul |
Split from OCA#843 as requested; this commit only migrates queue_job_cron_jobrunner.
|
@guewen, could we maybe merge this one please? |
|
/ocabot merge nobump |
|
What a great day to merge this nice PR. Let's do it! |
|
@guewen your merge command was aborted due to failed check(s), which you can inspect on this commit of 19.0-ocabot-merge-pr-843-by-guewen-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. |
Split from OCA#843 as requested; this commit only migrates queue_job_cron_jobrunner.
|
Hi @tishmen, the last Could you please rebase on 19.0 and review the tests that are failing so we can merge this? |
|
Hi @AndreuOForgeFlow. Your command failed:
Ocabot commands
More information
|
Scope
Depends on
Summary
Temporary references
This will be removed once the base PR merges.
Pre-commit
Tests