[14.0][MIG] report_csv: Migration to 14.0#511
Conversation
ferran-S73
force-pushed
the
14.0-mig-report_csv
branch
from
60fb4f1 to
197ccc2
Compare
ferran-S73
force-pushed
the
14.0-mig-report_csv
branch
from
197ccc2 to
3e0eed1
Compare
ferran-S73
force-pushed
the
14.0-mig-report_csv
branch
from
3e0eed1 to
060c2b3
Compare
|
@davejames could you review please? |
|
@OCA/reporting-engine-maintainers Could we merge this ? |
|
/ocabot merge nobump |
|
Hey, thanks for contributing! Proceeding to merge this for you. |
|
@etobella your merge command was aborted due to failed check(s), which you can inspect on this commit of 14.0-ocabot-merge-pr-511-by-etobella-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. |
|
Can you rebase? |
ferran-S73
force-pushed
the
14.0-mig-report_csv
branch
from
060c2b3 to
b1546b8
Compare
@etobella done |
|
tests fail might be unrelated, but can you reexecute pre-commit? |
It's working just fine on my local env |
|
Setup folder has not been added |
ferran-S73
force-pushed
the
14.0-mig-report_csv
branch
from
b1546b8 to
f2f016d
Compare
I've added the setup folder generated by pre-commit locally |
|
@etobella @adrienpeiffer @andreampiovesana @newtratip @OCA/reporting-engine-maintainers anyone has idea why is the test failing to donwload openjdk? |
|
There was an error with the CI Image. Now it has been fixed 😄 |
|
/ocabot merge nobump |
|
On my way to merge this fine PR! |
|
Congratulations, your PR was merged at 4f8d3c7. Thanks a lot for contributing to OCA. ❤️ |
16 participants
Original pull request #442
Updates and resolves conflicts