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

540 update dependencies #541

Merged
merged 2 commits into from
Jul 2, 2024
Merged

540 update dependencies #541

merged 2 commits into from
Jul 2, 2024

Conversation

dr0i
Copy link
Member

@dr0i dr0i commented Jun 21, 2024

Following #540.

Checked the repo using https://jeremylong.github.io/DependencyCheck/ . Updated the low hanging fruits. This decreased from:

Vulnerable Dependencies: 8 -> 7
Vulnerabilities Found: 12 -> 10

@dr0i dr0i marked this pull request as draft June 21, 2024 12:34
@dr0i dr0i mentioned this pull request Jun 21, 2024
11 tasks
@dr0i dr0i marked this pull request as ready for review June 21, 2024 12:47
@dr0i dr0i mentioned this pull request Jun 25, 2024
12 tasks
@dr0i
Copy link
Member Author

dr0i commented Jun 25, 2024

Hm, forgot to put it in release 6.1.0. So scheduled for 6.2.0.

@blackwinter
Copy link
Member

Hm, forgot to put it in release 6.1.0.

You said that these changes were part of the RC. But then they're not in the final release? Is the final release not based on the RC? If so, there's something wrong with the release process.

@dr0i
Copy link
Member Author

dr0i commented Jun 25, 2024

ACK. Not sure if https://github.com/metafacture/metafacture-core/wiki/Maintainer-Guidelines#making-a-release is wrong or not explicit enough. Will double check that when releasing #546 .

@dr0i dr0i self-assigned this Jun 25, 2024
@dr0i
Copy link
Member Author

dr0i commented Jul 1, 2024

I've updated the maintainer guidelines to fix missing -rc branch when releasing.

@dr0i dr0i merged commit f211233 into master Jul 2, 2024
5 checks passed
@dr0i dr0i mentioned this pull request Jul 15, 2024
12 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants