Skip to content

Conversation

@nohwnd
Copy link
Member

@nohwnd nohwnd commented Jan 29, 2025

Update the version so we can consume newer version in vstest and code coverage which depends on non-vulnerable system.text.json.

@nohwnd nohwnd requested a review from a team as a code owner January 29, 2025 16:27
@nohwnd
Copy link
Member Author

nohwnd commented Jan 29, 2025

In general what is the policy here about updating the packages? there are much newer versions 8.0.0 and 9.0.0 that we would also be able to use, and might prefer to use. When we updated vstest, few packages in here were older than expected. Can I just identify them and post PRs, or do you prefer to keep this repo with as few versions of packages as possible?

@nohwnd
Copy link
Member Author

nohwnd commented Jan 29, 2025

fyi @fhnaseer

@MichaelSimons
Copy link
Member

In general what is the policy here about updating the packages? there are much newer versions 8.0.0 and 9.0.0 that we would also be able to use, and might prefer to use. When we updated vstest, few packages in here were older than expected. Can I just identify them and post PRs, or do you prefer to keep this repo with as few versions of packages as possible?

You are free to open PRs and add as many versions that are needed. The goal of this repo is to support what ever versions the repos want to use via a self service model. The source-build team as maintainers will periodically remove unused packages so there is no need for repo owners to worry about cleanup.

@nohwnd
Copy link
Member Author

nohwnd commented Jan 30, 2025

Thank you!

@MichaelSimons MichaelSimons merged commit 5dc1b34 into dotnet:main Jan 30, 2025
4 checks passed
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.

2 participants