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

Update github workflows #5248

Closed
wants to merge 0 commits into from
Closed

Update github workflows #5248

wants to merge 0 commits into from

Conversation

matlads
Copy link
Contributor

@matlads matlads commented May 19, 2024

Description

Updates some github workflows to the latest versions

(...)

To Do

  • Documentation.
  • Changelog.
  • Tests.
  • CI

Copy link
Member

@snejus snejus left a comment

Choose a reason for hiding this comment

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

Added a couple of comments regarding python versions and testing

@@ -30,20 +30,3 @@ jobs:
- name: Check external links in docs
run: |
tox -e links

Copy link
Member

Choose a reason for hiding this comment

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

Why is this being removed?

with:
python-version: 3.9-dev
python-version: 3.12
Copy link
Member

Choose a reason for hiding this comment

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

I think it may be best to test the earliest supported version

@@ -12,16 +12,16 @@ jobs:
strategy:
matrix:
platform: [ubuntu-latest, windows-latest]
python-version: ['3.7', '3.8', '3.9', '3.x']
python-version: ['3.8', '3.9', '3.10', '3.11', '3.12']
Copy link
Member

Choose a reason for hiding this comment

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

Happy to test the most recent python versions here! We probably want to include a note in the changelog as well.

On the other hand, I think we want to continue testing 3.7 before it's been properly removed from the codebase, see the relevant ticket here: #5222

snejus added a commit that referenced this pull request Jun 5, 2024
## Description

Fixes #5222. 

Drop Python 3.7. `pyupgrade` is responsible for most of the changes in
the code. I undid some of the bits it attempted to update that aren't
strictly necessary:

1. Converting `List/Dict/Tuple` -> `list/dict/tuple` in modules that
have `from __future__ import annotations` import. This should be done in
a separate PR, and for all modules
2. Converting some `.format(` calls to f-strings. It didn't do it
consistently, and it should also be done in a separate PR, I believe.

Python upgrade unblocks several other PRs, for example #5266 and #5248.
@snejus
Copy link
Member

snejus commented Jun 5, 2024

@matlads Python 3.7 has now been dropped, so try updating your PR!

@Serene-Arc
Copy link
Contributor

Might be a tool PR? Sometimes people go through repos with automated tools to make easy commits and open PRs to try and boost their GitHub profile/resume. Showing that you've contributed to a popular repo, seem more active.

Might not be a bad idea to update some workflows though, as long as they don't break anything.

@matlads
Copy link
Contributor Author

matlads commented Jun 9, 2024

@matlads Python 3.7 has now been dropped, so try updating your PR!

Will do.

@Serene-Arc
Copy link
Contributor

Do you want to reopen this PR?

@matlads
Copy link
Contributor Author

matlads commented Jun 9, 2024

Do you want to reopen this PR?

No. I'll make a new one.

@Serene-Arc
Copy link
Contributor

Please don't. This has already been created, just reopen and commit changes.

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