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

[FIX] contract: removed dateutil from dependencies #1021

Merged

Conversation

suniagajose
Copy link

@suniagajose suniagajose commented Nov 8, 2023

Context
Since Odoo 16.0 has included dateutil on its dependencies, loading dateutil from here is generating Warning in Odoo.sh deployments

https://github.com/odoo/odoo/blob/16.0/requirements.txt#L46

Before
Odoo-sh-Builds-1-

After
Odoo-sh-Builds (2)

Since Odoo 16.0 has included dateutil on its dependencies,
loading dateutil from here is generating Warning in Odoo.sh
deployments
@suniagajose suniagajose force-pushed the 16.0-use-dateutil-from-odoo-suniagajose branch from 8db93e9 to 6c0f22e Compare November 8, 2023 19:42
@hugho-ad
Copy link

hugho-ad commented Nov 8, 2023

LGTM 👍

@suniagajose
Copy link
Author

hi @pedrobaeza

can you help me merging this, or let me know who is the maintener of this repository? i don't see anyone in the description

thanks,

cc @moylop260

Copy link

@desdelinux desdelinux left a comment

Choose a reason for hiding this comment

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

LGTM

@pedrobaeza
Copy link
Member

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 16.0-ocabot-merge-pr-1021-by-pedrobaeza-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit e78a02e into OCA:16.0 Nov 8, 2023
@OCA-git-bot
Copy link
Contributor

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

6 participants