Skip to content

python3.9+ #3042

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

Merged
merged 1 commit into from
Oct 28, 2023
Merged

python3.9+ #3042

merged 1 commit into from
Oct 28, 2023

Conversation

asottile
Copy link
Member

my hands are tied -- if you want to complain do it here: python/cpython#106532

@asottile asottile marked this pull request as ready for review October 28, 2023 18:35
@asottile asottile merged commit 762e681 into main Oct 28, 2023
@asottile asottile deleted the py39 branch October 28, 2023 18:35
Samyak2 added a commit to browser-history/browser-history that referenced this pull request May 5, 2024
@EwoutH
Copy link

EwoutH commented Aug 21, 2024

I don't know if you want to do at this stage, but if you want you could support Python 3.8 again:

EwoutH added a commit to EwoutH/solara that referenced this pull request Aug 21, 2024
Python 3.9 is the lowest version pre-commit supports. See pre-commit/pre-commit#3042
@asottile
Copy link
Member Author

I'm not, but it's good to see that I was right all along

maartenbreddels pushed a commit to widgetti/solara that referenced this pull request Aug 23, 2024
* pre-commit: Add pyupgrade

pyupgrade ensures the latest Python syntax supported is used.

* Run pre-commit

check yaml...............................................................Passed
fix end of files.........................................................Failed
- hook id: end-of-file-fixer
- exit code: 1
- files were modified by this hook

Fixing packages/solara-widget-manager8/tsconfig.json
Fixing packages/solara-widget-manager8/src
Fixing packages/solara-widget-manager8/style

trim trailing whitespace.................................................Passed
pyupgrade................................................................Failed
- hook id: pyupgrade
- exit code: 1
- files were modified by this hook

Rewriting solara/website/pages/doc_use_download.py

codespell................................................................Passed
ruff.....................................................................Passed
ruff-format..............................................................Passed
mypy.....................................................................Passed

* pre-commit: Update ruff version, use current repo

Use the latest ruff version (v0.6.1) from the current official ruff repo

* Use `is` for type comparisons

Fix ruff E721 Use `is` and `is not` for type comparisons, or `isinstance()` for isinstance checks

* Remove redefinition of unused `stack`

Fix ruff F811 in solara\toestand.py:148:40: F811 Redefinition of unused `stack` from line 147

* Run ruff on Jupyter Notebooks

Ruff now also works on Jupyter notebooks by default

* Revert "Run pre-commit"

This reverts commit f530ef8.

* CI: Update code-quality to use Python 3.9 and 3.12

Python 3.9 is the lowest version pre-commit supports. See pre-commit/pre-commit#3042

* Run pre-commit pyupgrade

* Fix pre-commit config warning in mypy hook

Fix: [WARNING] The 'exclude' field in hook 'mypy' is a regex, not a glob -- matching '/*' probably isn't what you want here

In regex, `.` matches any character except a newline, and `*` matches zero or more of the preceding element (in this case, any character). `.*` effectively replaces the `/*` glob pattern.

* Pre-commit: add check-toml, update mypy syntax
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants