Chore: inline Variables that immediately Returned#690
Chore: inline Variables that immediately Returned#690auvipy merged 3 commits intojpadilla:masterfrom yezz123:master
Conversation
yezz123
commented
Sep 29, 2021
- I just fix some issues related to code.
- Refactor some Parts of the code.
inline Variables that immediately Returned & Refactor Code Expression
auvipy
left a comment
There was a problem hiding this comment.
can you please update tests to see this changes wont regress?
It's not a new feature to resolve in the tests, cause this is just a simple way to refactor some parts of code, I didn't add any function that why 😀 |
|
I understand but still the changes are significant so you can add some additional test on existing tests only |
I don't see where we can add this test, cause this is just some simple changes that why 🤔 |
|
https://github.com/jpadilla/pyjwt/tree/master/tests tests for related modules can be found here |
|
@auvipy, |
|
+1 |
| if headers and "alg" in headers and headers["alg"]: | ||
| algorithm = headers["alg"] | ||
|
|
||
| if algorithm not in self._valid_algs: |
There was a problem hiding this comment.
may I know why this was removed?
There was a problem hiding this comment.
Because it just pass statement, it has no influence except time spent for if condition check