Skip to content

Conversation

Link9001
Copy link

Proposed Changes

Based on the Comment on my previous pull request, here is the update of the nuget package. It was only one.

Types of Changes

What types of changes does your code introduce to this project?
Put an x in the boxes that apply

  • Nuget package update
  • Bug fix (non-breaking change which fixes issue #NNNN)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause an observable behavior change in existing systems)
  • Documentation improvements (corrections, new content, etc)
  • Cosmetic change (whitespace, formatting, etc)

Checklist

Put an x in the boxes that apply. You can also fill these out after creating
the PR. If you're unsure about any of them, don't hesitate to ask on the
mailing list. We're here to help! This is simply a reminder of what we are
going to look for before merging your code.

  • I have read the CONTRIBUTING.md document
  • [?] I have signed the CA (see https://cla.pivotal.io/sign/rabbitmq)
  • All tests pass locally with my changes
  • [-] I have added tests that prove my fix is effective or that my feature works
  • [-] I have added necessary documentation (if appropriate)
  • [-] Any dependent changes have been merged and published in related repositories

Further Comments

@lukebakken
Copy link
Collaborator

@Link9001 - the System.Text.Json package should be updated to the lowest safe version, which appears to be 6.0.10 -

https://www.nuget.org/packages/System.Text.Json/6.0.10

@Link9001 Link9001 changed the title 6.x: Update nuget package System.Text.Json to 8.0.5 6.x: Update nuget package System.Text.Json to 6.0.10 Aug 19, 2025
@Link9001
Copy link
Author

Link9001 commented Aug 19, 2025

@Link9001 - the System.Text.Json package should be updated to the lowest safe version, which appears to be 6.0.10 -

https://www.nuget.org/packages/System.Text.Json/6.0.10

@lukebakken Yes you're absolutely right, I only looked top down. Thanks for checking twice.

Copy link
Collaborator

@lukebakken lukebakken left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@lukebakken lukebakken merged commit d4cb533 into rabbitmq:6.x Aug 19, 2025
2 of 4 checks passed
@lukebakken lukebakken added this to the 6.9.0 milestone Aug 19, 2025
@lukebakken lukebakken self-assigned this Aug 19, 2025
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