close
Skip to content

Drop Python 2 support - #346

Merged
pquentin merged 1 commit into
python-trio:masterfrom
bluetech:drop-py2
Aug 26, 2021
Merged

Drop Python 2 support#346
pquentin merged 1 commit into
python-trio:masterfrom
bluetech:drop-py2

Conversation

@bluetech

Copy link
Copy Markdown
Member

As discussed in #341, this drops Python 2 support. There's not big advantage to it, just makes the code a bit nicer (the typing in particular).

@codecov

codecov Bot commented Aug 25, 2021

Copy link
Copy Markdown

Codecov Report

Merging #346 (586b6fb) into master (9a3fd7e) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master      #346   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           10        10           
  Lines         1186      1169   -17     
  Branches        47        46    -1     
=========================================
- Hits          1186      1169   -17     
Impacted Files Coverage Δ
trustme/__main__.py 100.00% <ø> (ø)
trustme/__init__.py 100.00% <100.00%> (ø)
trustme/_cli.py 100.00% <100.00%> (ø)
trustme/tests/test_cli.py 100.00% <0.00%> (ø)
trustme/tests/test_trustme.py 100.00% <0.00%> (ø)
a/trustme/trustme/tests/test_cli.py 100.00% <0.00%> (ø)
a/trustme/trustme/tests/test_trustme.py 100.00% <0.00%> (ø)
...sers/runner/work/trustme/trustme/tests/test_cli.py 100.00% <0.00%> (ø)
.../runner/work/trustme/trustme/tests/test_trustme.py 100.00% <0.00%> (ø)

@bluetech
bluetech force-pushed the drop-py2 branch 2 times, most recently from ffa92eb to a325e38 Compare August 25, 2021 10:40

@pquentin pquentin left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for tackling this! The type annotations are much better.

Comment thread trustme/__init__.py

@pquentin pquentin left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks! Can you please add a news fragment?

@bluetech

Copy link
Copy Markdown
Member Author

I added a news fragment.

@pquentin
pquentin merged commit bdd78cb into python-trio:master Aug 26, 2021
@pquentin

Copy link
Copy Markdown
Member

Thanks! I had difficulties updating the required checks from my phone, hence the delay.

@bluetech
bluetech deleted the drop-py2 branch July 7, 2023 18:32
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.

2 participants