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] shopinvader_api_payment: fix dependency on sips #97

Merged
merged 1 commit into from
Jul 23, 2024

Conversation

simahawk
Copy link
Contributor

This dep is not needed in the base module.

@lmignon ping :)

This dep is not needed in the base module.
@lmignon
Copy link
Contributor

lmignon commented Jul 23, 2024

/ocabot merge patch

@shopinvader-git-bot
Copy link

@lmignon The merge process could not start, because command git fetch --quiet --force --prune https://github.com/shopinvader/odoo-shopinvader-payment 'refs/heads/*:refs/heads/*' failed with output:

fatal: detected dubious ownership in repository at '/app/run/.cache/oca-mqt/github.com/shopinvader/odoo-shopinvader-payment'
To add an exception for this directory, call:

	git config --global --add safe.directory /app/run/.cache/oca-mqt/github.com/shopinvader/odoo-shopinvader-payment

@sbidoul
Copy link
Member

sbidoul commented Jul 23, 2024

/ocabot merge patch

@shopinvader-git-bot
Copy link

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 16.0-ocabot-merge-pr-97-by-sbidoul-bump-patch, awaiting test results.

shopinvader-git-bot pushed a commit that referenced this pull request Jul 23, 2024
Signed-off-by sbidoul
@shopinvader-git-bot
Copy link

@sbidoul The merge process could not be finalized, because command twine check odoo_addon_shopinvader_api_payment-16.0.1.0.1-py3-none-any.whl failed with output:

Checking odoo_addon_shopinvader_api_payment-16.0.1.0.1-py3-none-any.whl: �[31mFAILED�[0m
�[31mERROR   �[0m `long_description` has syntax errors in markup and would not be        
         rendered on PyPI.                                                      
         No content rendered from RST source.                                   
�[33mWARNING �[0m `long_description_content_type` missing. defaulting to `text/x-rst`.   

@sbidoul
Copy link
Member

sbidoul commented Jul 23, 2024

This module had an empty README. I added one in 6a0e617.

/ocabot merge minor

@shopinvader-git-bot
Copy link

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

@shopinvader-git-bot shopinvader-git-bot merged commit b550fd5 into shopinvader:16.0 Jul 23, 2024
4 checks passed
@shopinvader-git-bot
Copy link

Congratulations, your PR was merged at 6869e39. Thanks a lot for contributing to shopinvader. ❤️

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