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

[12.0] FIX CI #286

Merged
merged 1 commit into from
Aug 11, 2021
Merged

[12.0] FIX CI #286

merged 1 commit into from
Aug 11, 2021

Conversation

ivantodorovich
Copy link

  • Missing requirements.txt for l10n_fr_chorus_account

@legalsylvain
Copy link
Contributor

thanks. fast merging.

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 12.0-ocabot-merge-pr-286-by-legalsylvain-bump-nobump, awaiting test results.

OCA-git-bot added a commit that referenced this pull request May 4, 2021
Signed-off-by legalsylvain
@OCA-git-bot
Copy link
Contributor

@legalsylvain your merge command was aborted due to failed check(s), which you can inspect on this commit of 12.0-ocabot-merge-pr-286-by-legalsylvain-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.

@ivantodorovich
Copy link
Author

Ok it seems it's more broken than I thought 😅
Anyway this PR does fix one error, so maybe you can merge with the button instead?

For the other errors... I don't have the time to fix module by module right now :/

@ivantodorovich
Copy link
Author

@alexis-via current ci errors are due to modules you maintain:

  • l10n_fr_business_document_import
  • l10n_fr_chorus_ubl

Can you check them up?

They seem to have originated here: #229
In the future please merge PRs with ocabot, so we can avoid this situation

@ivantodorovich
Copy link
Author

rebased :) we can still merge it I believe, just a tiny step forward but still

@legalsylvain
Copy link
Contributor

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 12.0-ocabot-merge-pr-286-by-legalsylvain-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 902b0e9 into OCA:12.0 Aug 11, 2021
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 902b0e9. 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.

3 participants