Skip to content

Draft: Upgrade to pyee>=9.0.0#1587

Closed
jean-edouard-boulanger wants to merge 3 commits into
microsoft:mainfrom
jean-edouard-boulanger:upgrade-pyee-to-9-0-4
Closed

Draft: Upgrade to pyee>=9.0.0#1587
jean-edouard-boulanger wants to merge 3 commits into
microsoft:mainfrom
jean-edouard-boulanger:upgrade-pyee-to-9-0-4

Conversation

@jean-edouard-boulanger
Copy link
Copy Markdown

Python playwright uses an old-ish version of pyee (8.1.0) - this PR upgrades to >=9.0.0. There was a previous attempt here but was never merged because of pyee >=9.0.0 was not available in conda at that time. This is no longer the case:

❯ conda search -f pyee
Loading channels: done
# Name                       Version           Build  Channel
...
pyee                           8.0.1    pyh9f0ad1d_0  conda-forge
pyee                           8.1.0    pyh9f0ad1d_0  conda-forge
pyee                           8.1.0    pyhd8ed1ab_0  conda-forge
pyee                           9.0.2    pyhd8ed1ab_0  conda-forge
pyee                           9.0.4    pyhd8ed1ab_0  conda-forge

No regression between main and this branch.

Thanks for taking a look 🙏

@jean-edouard-boulanger
Copy link
Copy Markdown
Author

@microsoft-github-policy-service agree [company="default"]

@jean-edouard-boulanger
Copy link
Copy Markdown
Author

@microsoft-github-policy-service agree

@jean-edouard-boulanger jean-edouard-boulanger changed the title Upgrade to pyee>=9.0.0 Draft: Upgrade to pyee>=9.0.0 Oct 11, 2022
@jean-edouard-boulanger
Copy link
Copy Markdown
Author

I see a couple of pyright failures (only happen with pyee>=9.0.0). Fixing

Comment thread meta.yaml
- python
- greenlet ==1.1.3
- pyee ==8.1.0
- pyee >=9.0.0
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.

Suggested change
- pyee >=9.0.0
- pyee ==9.0.4

let's keep that? In the past they broke us with a minor version bump.

Comment thread setup.py
install_requires=[
"greenlet==1.1.3",
"pyee==8.1.0",
"pyee>=9.0.0",
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.

ditto

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants