Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

replace cgi#139

Merged
jschlyter merged 3 commits intoIdentityPython:mainIdentityPython/JWTConnect-Python-CryptoJWT:mainfrom
jschlyter:replace_cgijschlyter/cryptojwt:replace_cgiCopy head branch name to clipboard
May 1, 2023
Merged

replace cgi#139
jschlyter merged 3 commits intoIdentityPython:mainIdentityPython/JWTConnect-Python-CryptoJWT:mainfrom
jschlyter:replace_cgijschlyter/cryptojwt:replace_cgiCopy head branch name to clipboard

Conversation

@jschlyter
Copy link
Collaborator

@jschlyter jschlyter commented Apr 28, 2023

CGI module is deprecated since Python 3.11, replace it. More information at https://peps.python.org/pep-0594/#cgi

depends on #138

@jschlyter jschlyter requested a review from rohe April 28, 2023 06:34
@codecov-commenter
Copy link

Codecov Report

Merging #139 (d3a66f4) into main (8cb2034) will increase coverage by 0.01%.
The diff coverage is 100.00%.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@            Coverage Diff             @@
##             main     #139      +/-   ##
==========================================
+ Coverage   77.44%   77.45%   +0.01%     
==========================================
  Files          42       42              
  Lines        4362     4364       +2     
  Branches      982      907      -75     
==========================================
+ Hits         3378     3380       +2     
- Misses        717      718       +1     
+ Partials      267      266       -1     
Impacted Files Coverage Δ
src/cryptojwt/utils.py 82.55% <100.00%> (+0.23%) ⬆️

... and 1 file with indirect coverage changes

@peppelinux peppelinux self-requested a review May 1, 2023 08:15
@jschlyter jschlyter merged commit 7065213 into IdentityPython:main May 1, 2023
@jschlyter jschlyter deleted the replace_cgi branch May 1, 2023 13:27
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.

4 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.