Skip to content

Type hints based on #61 #123

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

Closed

Conversation

lucywang000
Copy link
Member

This pr is based on @kmike 's work in #61.

I wanted to set the pr target to the type-hint branch but since it has conflicts with master I'm not able to do that. So I have to set the pr target branch to master.

@codecov
Copy link

codecov bot commented Jan 30, 2019

Codecov Report

Merging #123 into master will increase coverage by 0.23%.
The diff coverage is 98.64%.

@@            Coverage Diff             @@
##           master     #123      +/-   ##
==========================================
+ Coverage    95.3%   95.54%   +0.23%     
==========================================
  Files           7        8       +1     
  Lines         469      494      +25     
  Branches       93       93              
==========================================
+ Hits          447      472      +25     
  Misses         15       15              
  Partials        7        7
Impacted Files Coverage Δ
w3lib/_types.py 100% <100%> (ø)
w3lib/html.py 93.85% <100%> (+0.05%) ⬆️
w3lib/http.py 100% <100%> (ø) ⬆️
w3lib/url.py 97.95% <100%> (+0.04%) ⬆️
w3lib/encoding.py 100% <100%> (ø) ⬆️
w3lib/util.py 71.42% <66.66%> (+1.73%) ⬆️

@codecov
Copy link

codecov bot commented Jan 30, 2019

Codecov Report

Merging #123 into master will increase coverage by 0.23%.
The diff coverage is 98.63%.

@@            Coverage Diff             @@
##           master     #123      +/-   ##
==========================================
+ Coverage    95.3%   95.54%   +0.23%     
==========================================
  Files           7        8       +1     
  Lines         469      494      +25     
  Branches       93       93              
==========================================
+ Hits          447      472      +25     
  Misses         15       15              
  Partials        7        7
Impacted Files Coverage Δ
w3lib/_types.py 100% <100%> (ø)
w3lib/html.py 93.85% <100%> (+0.05%) ⬆️
w3lib/http.py 100% <100%> (ø) ⬆️
w3lib/url.py 97.95% <100%> (+0.04%) ⬆️
w3lib/encoding.py 100% <100%> (ø) ⬆️
w3lib/util.py 71.42% <66.66%> (+1.73%) ⬆️

@lucywang000
Copy link
Member Author

@kmike could you please take a look?

@lucywang000
Copy link
Member Author

lucywang000 commented Jun 9, 2021

Closed. There are quite some changes on the master branch, especially python 2 support is dropped. New pr in #172 .

@lucywang000 lucywang000 closed this Jun 9, 2021
@lucywang000 lucywang000 mentioned this pull request Jun 10, 2021
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