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

Pin blinker to version <1.8 as fix for #329. #330

Closed
wants to merge 2 commits into from
Closed

Pin blinker to version <1.8 as fix for #329. #330

wants to merge 2 commits into from

Conversation

mpibpc-mroose
Copy link

As PyleniumIO relies on selenium-wire (which is no longer alive/maintained) and this project again uses blinker one would like to pin this "stack" to a working version.

selenium-wirein it's final version uses a private method of blinker which has been removed in the 1.8 release. As selenium-wire cannot be changed the only way is to pin blinker.

As PyleniumIO relies on selenium-wire (which is no longer alive) and this project again on blinker one would like to pin it to a working version. seleium-wire uses a private method of blinker which has been removed in the 1.8 release. As selenium-wire cannot be changed the only way is to pin blinker.
@ElSnoMan
Copy link
Owner

ElSnoMan commented May 1, 2024

You've raised really good points! It's probably best to remove selenium-wire as a dependency. If someone wants to add similar functionality, they are welcome to!

@LostInDarkMath
Copy link
Contributor

Yes I agree. Removing the selenium-wire dependency would be the cleanest solution.

@ElSnoMan
Copy link
Owner

I'm closing this as we've now removed selenium-wire completely. This will be out in v1.21

@ElSnoMan ElSnoMan closed this May 24, 2024
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.

3 participants