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

Dependency/Project updates #166

Merged
merged 11 commits into from
Aug 21, 2023
Merged

Conversation

derkweijers
Copy link
Contributor

Ahoj,

While still in the spirits of EuroPython 2023, I was checking out the Pyvo website/code and noticed some old dependencies and CVEs.
This PR updates the dependencies and fixes the czech_holiday package since it's functions have changed a bit. The before_first_request decorator is removed in Flask 2.3, this code is moved to the create_app function itself.

All tests (tested on Python 3.9, 3.10 & 3.11) are working. Please note that czech_holiday can't be used for dates before 2016, so these are excluded from the spider.

The classifiers/packages in setup.py are updated. It's been a while since I've worked with this module, so please let me know if I've missed something.

Finally, the GitHub action is updated to run the tests with Python 3.9, 3.10 and 3.11.

@hroncok
Copy link
Member

hroncok commented Jul 26, 2023

Thank you @derkweijers! I approved the workflow but will let @encukou do the merging (I cannot fix problems with the deployment if they happen).

@encukou
Copy link
Member

encukou commented Aug 21, 2023

Thanks for the PR, and sorry for the delay!

I'd like to keep the old version of czech_holidays, since we do use years before 2016.

It seems rosti uses Python 3.8, so I'll add that.

And, note that the CVEs dependabot opened don't really apply here :)

@encukou encukou merged commit 251fa3a into pyvec:master Aug 21, 2023
4 checks passed
@encukou
Copy link
Member

encukou commented Aug 21, 2023

It seems rosti uses Python 3.8, so I'll add that.

Turns out that was outdated info! The live site is now on Python 3.11 :)
Thanks again for the PR!

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