Skip to content
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] module_auto_update #2543

Merged
merged 6 commits into from
Apr 13, 2023

Conversation

fkantelberg
Copy link
Member

This migrates the module to 16.0. No changes big changes were necessary only a deprecation warning in the flush.

@fkantelberg fkantelberg force-pushed the 16.0-mig-module_auto_update branch from 5205a5e to 3186d55 Compare February 7, 2023 13:54
@fkantelberg fkantelberg force-pushed the 16.0-mig-module_auto_update branch from 3186d55 to 5637090 Compare February 9, 2023 07:07
@simahawk
Copy link
Contributor

simahawk commented Mar 7, 2023

/ocabot migration module_auto_update

@OCA-git-bot OCA-git-bot added this to the 16.0 milestone Mar 7, 2023
@simahawk
Copy link
Contributor

simahawk commented Mar 7, 2023

Seems we have a flappy test in base_exception.

@hbrunn
Copy link
Member

hbrunn commented Apr 12, 2023

/ocabot rebase

@OCA-git-bot
Copy link
Contributor

@hbrunn The rebase process failed, because command git push --force initOS tmp-pr-2543:16.0-mig-module_auto_update failed with output:

remote: Permission to initOS/server-tools.git denied to OCA-git-bot.
fatal: unable to access 'https://github.com/initOS/server-tools/': The requested URL returned error: 403

@hbrunn
Copy link
Member

hbrunn commented Apr 12, 2023

@fkantelberg could you rebase?

(if you allow edits by maintainers when opening the PR the above command works and things go smoother)

@fkantelberg fkantelberg force-pushed the 16.0-mig-module_auto_update branch from 5637090 to f555e90 Compare April 13, 2023 05:58
@fkantelberg
Copy link
Member Author

@hbrunn rebased and I'll set the flag the next time. Please try again.

@hbrunn
Copy link
Member

hbrunn commented Apr 13, 2023

@fkantelberg thanks, and as @yostashiro's comment was fixed too I'll say

/ocabot merge nobump

@hbrunn
Copy link
Member

hbrunn commented Apr 13, 2023

/ocabot merge nobump

@OCA-git-bot
Copy link
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). 🤖

@OCA-git-bot
Copy link
Contributor

@hbrunn The merge process could not start, because command git push origin 16.0-ocabot-merge-pr-2543-by-hbrunn-bump-nobump failed with output:

remote: Resolving deltas:   0% (0/5)        
remote: Resolving deltas:  20% (1/5)        
remote: Resolving deltas:  40% (2/5)        
remote: Resolving deltas:  60% (3/5)        
remote: Resolving deltas:  80% (4/5)        
remote: Resolving deltas: 100% (5/5)        
remote: Resolving deltas: 100% (5/5), done.        
To https://github.com/OCA/server-tools
 ! [remote rejected]     16.0-ocabot-merge-pr-2543-by-hbrunn-bump-nobump -> 16.0-ocabot-merge-pr-2543-by-hbrunn-bump-nobump (cannot lock ref 'refs/heads/16.0-ocabot-merge-pr-2543-by-hbrunn-bump-nobump': reference already exists)
error: failed to push some refs to 'https://***@github.com/OCA/server-tools'

@OCA-git-bot
Copy link
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 16.0-ocabot-merge-pr-2543-by-hbrunn-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit c8a05e8 into OCA:16.0 Apr 13, 2023
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at c1177ac. Thanks a lot for contributing to OCA. ❤️

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.

7 participants