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

Bump HIC.BadMedicine and Microsoft.Bcl.AsyncInterfaces #1824

Closed

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github May 16, 2024

Bumps HIC.BadMedicine and Microsoft.Bcl.AsyncInterfaces. These dependencies needed to be updated together.
Updates HIC.BadMedicine from 1.2.1 to 1.2.2

Release notes

Sourced from HIC.BadMedicine's releases.

v1.2.2

What's Changed

Full Changelog: HicServices/SynthEHR@v1.2.1...v1.2.2

Changelog

Sourced from HIC.BadMedicine's changelog.

[1.2.2] - 2024-05-16

-Add warning about naming deprecation, see README

Commits
  • 48d3ad6 Bump CsvHelper from 32.0.2 to 32.0.3
  • 6919fe8 bump to 1.2.2 and add deprication warning (#196)
  • 01d3f87 Bump YamlDotNet from 15.1.2 to 15.1.4
  • 6de7d7b Bump CsvHelper from 32.0.1 to 32.0.2
  • 1fc9feb Bump CsvHelper from 31.0.4 to 32.0.1
  • 97d7f98 Bump NUnit.Analyzers from 4.1.0 to 4.2.0
  • 67e72bc Bump CsvHelper from 31.0.3 to 31.0.4
  • 82e2319 Bump CsvHelper from 31.0.2 to 31.0.3
  • See full diff in compare view

Updates Microsoft.Bcl.AsyncInterfaces from 6.0.0 to 7.0.0

Release notes

Sourced from Microsoft.Bcl.AsyncInterfaces's releases.

.NET 7.0.0

Release

.NET 7.0 RC 2

Release

.NET 7.0 RC 1

Release

.NET 7.0 Preview 7

Release

.NET 7.0 Preview 6

Release

.NET 7.0 Preview 5

Release

.NET 7.0 Preview 4

Release

.NET 7.0 Preview 3

Release

.NET 7.0 Preview 2

Release

.NET 6.0.30

Release

.NET 6.0.29

Release

.NET 6.0.27

Release

.NET 6.0.26

Release

.NET 6.0.25

Release

What's Changed

... (truncated)

Commits
  • d099f07 Merge in 'release/7.0' changes
  • 0cee4aa [release/7.0] Fix loading app-local ICU (#77118)
  • bebb595 Merge in 'release/7.0' changes
  • cd2d837 Port stackoverflow fix from Roslyn to SourceGenerator PolyFill (#76954)
  • db34250 Merge in 'release/7.0' changes
  • f252bc6 Merged PR 26705: Empty to commit to diverge internal and public branches
  • 075e74e [mono][s390x] Fix wrong implementation of OP_CHECK_THIS (#76920)
  • aacc243 Update macos-11 vmImage to macos-12 for XCode 14. (#76973)
  • c7b59fc [release/7.0] Reintroduce typo in ICorDebug public API names (#76967)
  • 156aa8a Merge in 'release/7.0' changes
  • Additional commits viewable in compare view

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

Bumps [HIC.BadMedicine](https://github.com/HicServices/BadMedicine) and [Microsoft.Bcl.AsyncInterfaces](https://github.com/dotnet/runtime). These dependencies needed to be updated together.

Updates `HIC.BadMedicine` from 1.2.1 to 1.2.2
- [Release notes](https://github.com/HicServices/BadMedicine/releases)
- [Changelog](https://github.com/HicServices/BadMedicine/blob/main/CHANGELOG.md)
- [Commits](HicServices/SynthEHR@v1.2.1...v1.2.2)

Updates `Microsoft.Bcl.AsyncInterfaces` from 6.0.0 to 7.0.0
- [Release notes](https://github.com/dotnet/runtime/releases)
- [Commits](dotnet/runtime@v6.0.0...v7.0.0)

---
updated-dependencies:
- dependency-name: HIC.BadMedicine
  dependency-type: direct:production
  update-type: version-update:semver-patch
- dependency-name: Microsoft.Bcl.AsyncInterfaces
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
@dependabot dependabot bot added .NET Pull requests that update .net code dependencies Pull requests that update a dependency file labels May 16, 2024
@dependabot dependabot bot requested review from jas88 and JFriel May 16, 2024 00:24
Copy link
Contributor Author

dependabot bot commented on behalf of github May 16, 2024

Looks like these dependencies are no longer a dependency, so this is no longer needed.

@dependabot dependabot bot closed this May 16, 2024
@dependabot dependabot bot deleted the dependabot/nuget/develop/multi-4bc9f0140c branch May 16, 2024 08:15
@jas88
Copy link
Member

jas88 commented May 16, 2024

@JFriel the obsolete AsyncInterfaces one was noted as "Add Microsoft.Bcl.AsyncInterfaces 6.0.0 for plugin dependancy tree", Dependabot seems to think it's somehow related to BadMedicine/SynthEHR and the new version needs 7.0.0 not 6.0.0?

@JFriel
Copy link
Collaborator

JFriel commented May 20, 2024

@JFriel the obsolete AsyncInterfaces one was noted as "Add Microsoft.Bcl.AsyncInterfaces 6.0.0 for plugin dependancy tree", Dependabot seems to think it's somehow related to BadMedicine/SynthEHR and the new version needs 7.0.0 not 6.0.0?

It's odd it thinks it's SynthEHR related. It's actually backfilling for fo-dicom, which still expects 6.0.0. Don't think we want to bump Microsoft.Bcl.AsyncInterfaces, so once fo-dicom is updated we should be able to just remove this dependancy

@jas88
Copy link
Member

jas88 commented May 20, 2024

@JFriel ah yes - fo-dicom has that as an explicit dependency, so us referencing fo-dicom directly should avoid the need for that. (Plus we're bundling RdmpDicom as a plugin anyway, which already has that as a dependency.)

@JFriel
Copy link
Collaborator

JFriel commented May 20, 2024

@jas88 This might need a bit of investigation, as I've just tried removing the Microsoft.Bcl.AsyncInterfaces from the RDMP.Core dependencies and it causes issues with Microsoft.Bcl.AsyncInterfaces not being available

@jas88
Copy link
Member

jas88 commented May 20, 2024

@JFriel Did you still get that issue after replacing the AsyncInterfaces reference with a reference to fo-dicom? The latter should automatically include AsyncInterfaces for you, so if that doesn't work something is broken somewhere in nupkg land...

@JFriel
Copy link
Collaborator

JFriel commented May 20, 2024

@jas88 No, just removed AsyncInterface without replacing it. Not sure we should be including a reference to fo-dicom in Rdmp.Core , but may be part of the wider plugin conversation

@jas88
Copy link
Member

jas88 commented May 20, 2024

@JFriel We're including fo-dicom anyway, but #1835 should be a cleaner fix for this - RdmpDicom's plugin package will now include its own copy again, no longer relying on RDMP having a copy (which it used to, but doesn't any more).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file .NET Pull requests that update .net code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants