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

Update dependencies #216

Closed
wants to merge 3 commits into from
Closed

Update dependencies #216

wants to merge 3 commits into from

Conversation

jpwgnr
Copy link

@jpwgnr jpwgnr commented Oct 18, 2023

Fix for 2 high and 4 moderate vulnerabilities.

  • poetry update
  • fix needed for error same as here: solved by fixing flask for now <2.2.5
  • manually going to pycov>4.0 to prevent warning shown here
  • fix needed for werkzeug=3.0.0 same as here: solved by fixing werkzeug for now <3.0.0

Only ran tests, not sure how good coverage is. So please check if important functionality is broken,

@jpwgnr jpwgnr requested a review from maxnoe October 18, 2023 11:15
@codecov-commenter
Copy link

Codecov Report

Merging #216 (d4fad2b) into main (2a4487c) will not change coverage.
The diff coverage is n/a.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

@@           Coverage Diff           @@
##             main     #216   +/-   ##
=======================================
  Coverage   77.83%   77.83%           
=======================================
  Files          23       23           
  Lines        1060     1060           
=======================================
  Hits          825      825           
  Misses        235      235           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

jpwgnr pushed a commit that referenced this pull request Oct 18, 2023
@maxnoe maxnoe mentioned this pull request Dec 11, 2023
@maxnoe
Copy link
Member

maxnoe commented Dec 11, 2023

superseded by #219

@maxnoe maxnoe closed this Dec 11, 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