Changed the annotated type of data parameter in the encode and decode method#204
Changed the annotated type of data parameter in the encode and decode method#204subhajitsaha01 wants to merge 2 commits intokjd:masterfrom
Conversation
|
Hi @kjd is there anything I could do to get this PR resolved |
Well, passing the build tests would be a good start. The PR didn't look like it was written by a human so did not spend further time on it. |
The Build tests are failing because the union of types are allowed after python version 3.10 and thus are failing for python versions 3.8 and 3.9. I need to change the format of type Do tell me if I should proceed with the same? |
In the test_idna.py file, for the
dataparameter of thedecodemethod of theCodecclass, a string value has been passed and accepted but the annotated type is onlybytes. I have changed the static annotated type tostr|bytes.Links for the test file line, where the string values are passed to the
dataparameter of thedecodemethod of the Codec class :idna/tests/test_idna.py
Line 267 in cde1d8a
idna/tests/test_idna.py
Line 271 in cde1d8a
idna/tests/test_idna.py
Line 275 in cde1d8a
In the test_idna.py file, for the
dataparameter of theencodemethod of theCodecclass, a bytes value has been passed and accepted but the annotated type is onlystr. I have changed the static annotated type tostr|bytes.Link for the test file line, where the bytes values are passed to the
dataparameter of theencodemethod of the Codec class :idna/tests/test_idna.py
Line 256 in cde1d8a