-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Drop support for Python 3.8 #9774
Changes from all commits
79a7b1e
d83fb24
83244c0
833762f
33cd7b7
bd400b5
7a089b5
4976d7c
2b1bbed
1d066c5
77d72d2
b0f9ce0
0b33db5
4b45192
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -30,7 +30,7 @@ jobs: | |
timeout-minutes: 5 | ||
strategy: | ||
matrix: | ||
python-version: [3.8, 3.9, "3.10", "3.11", "3.12"] | ||
python-version: [3.9, "3.10", "3.11", "3.12"] | ||
outputs: | ||
python-key: ${{ steps.generate-python-key.outputs.key }} | ||
steps: | ||
|
@@ -72,7 +72,7 @@ jobs: | |
needs: prepare-tests-linux | ||
strategy: | ||
matrix: | ||
python-version: [3.8, 3.9, "3.10", "3.11", "3.12"] | ||
python-version: [3.9, "3.10", "3.11", "3.12"] | ||
steps: | ||
- name: Check out code from GitHub | ||
uses: actions/[email protected] | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -38,7 +38,7 @@ jobs: | |
timeout-minutes: 45 | ||
strategy: | ||
matrix: | ||
python-version: ["3.8", "3.12"] | ||
python-version: ["3.9", "3.12"] | ||
batches: [4] | ||
batchIdx: [0, 1, 2, 3] | ||
steps: | ||
|
@@ -198,7 +198,7 @@ jobs: | |
echo ${{ github.event.pull_request.number }} | tee pr_number.txt | ||
- name: Upload PR number | ||
if: | ||
startsWith(steps.python.outputs.python-version, '3.8') && matrix.batchIdx == 0 | ||
startsWith(steps.python.outputs.python-version, '3.9') && matrix.batchIdx == 0 | ||
uses: actions/[email protected] | ||
with: | ||
name: pr_number | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -31,7 +31,7 @@ jobs: | |
strategy: | ||
fail-fast: false | ||
matrix: | ||
python-version: [3.8, 3.9, "3.10", "3.11", "3.12"] | ||
python-version: [3.9, "3.10", "3.11", "3.12"] | ||
outputs: | ||
python-key: ${{ steps.generate-python-key.outputs.key }} | ||
steps: | ||
|
@@ -175,7 +175,7 @@ jobs: | |
strategy: | ||
fail-fast: false | ||
matrix: | ||
python-version: [3.8, 3.9, "3.10", "3.11", "3.12"] | ||
python-version: [3.9, "3.10", "3.11", "3.12"] | ||
steps: | ||
- name: Set temp directory | ||
run: echo "TEMP=$env:USERPROFILE\AppData\Local\Temp" >> $env:GITHUB_ENV | ||
|
@@ -225,7 +225,7 @@ jobs: | |
fail-fast: false | ||
matrix: | ||
# We only run on the oldest supported version on Mac | ||
python-version: [3.8] | ||
python-version: [3.9] | ||
steps: | ||
- name: Check out code from GitHub | ||
uses: actions/[email protected] | ||
|
@@ -269,7 +269,7 @@ jobs: | |
strategy: | ||
fail-fast: false | ||
matrix: | ||
python-version: ["pypy-3.8", "pypy-3.9", "pypy-3.10"] | ||
python-version: ["pypy-3.9", "pypy-3.10"] | ||
steps: | ||
- name: Check out code from GitHub | ||
uses: actions/[email protected] | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,6 @@ | ||
Remove support for launching pylint with Python 3.8. | ||
Code that supports Python 3.8 can still be linted with the ``--py-version=3.8`` setting. | ||
|
||
``--min_pyver_end_position`` in the functional test runner is no longer relevant and is removed. | ||
|
||
Refs #9774 |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -41,7 +41,6 @@ class MethodArgsChecker(BaseChecker): | |
"positional-only-arguments-expected", | ||
"Emitted when positional-only arguments have been passed as keyword arguments. " | ||
"Remove the keywords for the affected arguments in the function call.", | ||
{"minversion": (3, 8)}, | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Shouldn't we replace this with a py-version check ? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Hmm, but if my library supports 3.7 - 3.12, and I'm using a 3.8 function with positional-only arguments in the wrong manner, wouldn't I rather continue to see the message even if There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I thought There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I agree with Jacob's second comment. I don't think it will add much benefit to spend the time to add support for that edge case. Removing this line is fine. |
||
), | ||
} | ||
options = ( | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure we should remove this, sometime there's a new column/line in a new version. In particular when a bug is fixed or a nodes end is adjusted (in cpython). When this is the case we choose the newer version. This is still true in new interpreters. (Introduced when dealing with 3.9 only column change afair ?)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the explanation, will revert.