Skip to content

[MIG][15.0] utm: migration script - #3282

Merged
legalsylvain merged 1 commit into
OCA:15.0from
sang250399:v15_mig_utm
Jul 6, 2022
Merged

legalsylvain merged 1 commit into
OCA:15.0from
sang250399:v15_mig_utm

Conversation

@sang250399

Copy link
Copy Markdown

This PR

Migration done

@legalsylvain

This comment was marked as duplicate.

@OCA-git-bot OCA-git-bot added this to the 15.0 milestone Jun 15, 2022
@legalsylvain

Copy link
Copy Markdown
Contributor

/ocabot migration utm

@legalsylvain legalsylvain left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks.

@OCA-git-bot

Copy link
Copy Markdown
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). 🤖

@legalsylvain
legalsylvain merged commit 639392e into OCA:15.0 Jul 6, 2022
@pedrobaeza

Copy link
Copy Markdown
Member

@legalsylvain please wait until we resolve the CI problem (tests are not being executed) for merging new PRs.

@pedrobaeza

Copy link
Copy Markdown
Member

@legalsylvain you can continue, but please check before merging if the modules are being detected correctly (check in the logs for missing dependency text)

@pedrobaeza

Copy link
Copy Markdown
Member

In fact, in this PR the test haven't been correctly executed. Check in https://github.com/OCA/OpenUpgrade/runs/6892985524?check_suite_focus=true#step:13:54 that only base test has been run.

@pedrobaeza

Copy link
Copy Markdown
Member

But on main branch tests they are: https://github.com/OCA/OpenUpgrade/runs/7210598841?check_suite_focus=true#step:13:54

I'm totally puzzled about what's happening...

@legalsylvain

Copy link
Copy Markdown
Contributor

@legalsylvain please wait until we resolve the CI problem (tests are not being executed) for merging new PRs.

sorry I've missed that point. is there a dedicated issue for that trouble ?

@pedrobaeza

Copy link
Copy Markdown
Member

Please follow the conversation on mail PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants