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.

Sign up for GitHub

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

Jump to bottom

Update main dependencies #431

Merged
merged 2 commits into from
Sep 5, 2023
Merged

Update main dependencies #431

merged 2 commits into from
Sep 5, 2023

Conversation

yvanzo
Copy link
Contributor

@yvanzo yvanzo commented Sep 5, 2023 •
edited
Loading

Update main dependencies

  • Bump Node.js version from 16 (which will reach EOL within a week) to 18 (active LTS)
  • Bump ReportLab version from 3.6.12 to 3.6.13 (which fixes a vulnerability)

Draft in progress

  • Deploy to the test instance
  • Test it manually
  • Check if more dependencies can be updated at the same time

@github-actions

This comment has been minimized.

Sign in to view
@yvanzo
Copy link
Contributor Author

yvanzo commented Sep 5, 2023

@mayhem tested the admin part a bit.
I checked for other updates but there is no rush for those (no other security alert or upcoming EOL) and little time at the moment, thus merging as-is.

@yvanzo yvanzo marked this pull request as ready for review September 5, 2023 13:32
@yvanzo yvanzo merged commit b00057a into master Sep 5, 2023
2 checks passed
@yvanzo yvanzo deleted the updeps branch September 5, 2023 13:32
@github-actions
Copy link

github-actions bot commented Sep 5, 2023

Unit Test Results

  1 files  ±0    1 suites  ±0   14s ⏱️ ±0s
67 tests ±0  67 ✔️ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit b00057a. ± Comparison against base commit b00057a.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mayhem mayhem mayhem approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@yvanzo @mayhem