Skip to content
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

CVE-2023-36053: Updated the Django requirement #6

Merged
merged 10 commits into from
Aug 1, 2023

Conversation

hkage
Copy link

@hkage hkage commented Jul 26, 2023

In addition to the updated Django version, the jira requirement had to be pinned to <3.1 as the library changes the constructor parameter order (text and status_code) of the JIRAError from version 3.0.1 to 3.10.

I also removed tox-factor and the end-of-life Python versions 3.6 and 3.7 from the Github actions.

Testing

For testing you should install planning-poker-jira as described in the README and configure it with an exisiting JIRA server.

@hkage hkage changed the base branch from main to development July 26, 2023 14:30
@codecov-commenter
Copy link

codecov-commenter commented Jul 27, 2023

Codecov Report

Merging #6 (3c424d9) into development (23a4f4d) will not change coverage.
The diff coverage is n/a.

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

@@              Coverage Diff              @@
##           development        #6   +/-   ##
=============================================
  Coverage       100.00%   100.00%           
=============================================
  Files                7         7           
  Lines              218       218           
=============================================
  Hits               218       218           
Flag Coverage Δ
unittests 100.00% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

@hkage hkage marked this pull request as ready for review July 31, 2023 13:44
@W1ldPo1nter
Copy link
Contributor

LGTM. See comment in the main project.

@W1ldPo1nter W1ldPo1nter merged commit c8ab2d8 into development Aug 1, 2023
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.

3 participants