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

Develop #59

Merged
merged 40 commits into from
Aug 17, 2020
Merged

Develop #59

merged 40 commits into from
Aug 17, 2020

Conversation

jas88
Copy link
Member

@jas88 jas88 commented Aug 14, 2020

  • Fully adopt new FAnsiSql version (remove accidental dependency on old MySql namespacing)
  • New PACS client code to be more permissive of file formats
  • Add ProcessBasedCacheSource for pluggable PACS fetcher tools

jas88 and others added 30 commits July 6, 2020 16:15
* Bump HIC.RDMP.Plugin from 4.1.4 to 4.1.5
…le.Protobuf-3.12.4

Bump Google.Protobuf from 3.12.3 to 3.12.4
Bumps [Microsoft.NET.Test.Sdk](https://github.com/microsoft/vstest) from 16.6.1 to 16.7.0.
- [Release notes](https://github.com/microsoft/vstest/releases)
- [Commits](microsoft/vstest@v16.6.1...v16.7.0)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
…osoft.NET.Test.Sdk-16.7.0

Bump Microsoft.NET.Test.Sdk from 16.6.1 to 16.7.0
Bumps [System.Data.SqlClient](https://github.com/dotnet/corefx) from 4.8.1 to 4.8.2.
- [Release notes](https://github.com/dotnet/corefx/releases)
- [Commits](https://github.com/dotnet/corefx/commits)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
Bump System.Data.SqlClient from 4.8.1 to 4.8.2
jas88 and others added 7 commits August 14, 2020 10:15
Allow the PACS to send us lossy-compressed images or videos if it has them
…RDMP.Plugin.Test-4.1.7

Bump HIC.RDMP.Plugin.Test from 4.1.6 to 4.1.7
…RDMP.Plugin.UI-4.1.7

Bump HIC.RDMP.Plugin.UI from 4.1.6 to 4.1.7
@jas88 jas88 requested a review from tznind August 14, 2020 15:41
@lgtm-com
Copy link

lgtm-com bot commented Aug 14, 2020

This pull request fixes 11 alerts when merging 5089e24 into f6901d6 - view on LGTM.com

fixed alerts:

  • 4 for Useless assignment to local variable
  • 4 for Missing Dispose call on local IDisposable
  • 2 for Dereferenced variable may be null
  • 1 for Possible loss of precision

@tznind
Copy link
Contributor

tznind commented Aug 15, 2020

Looks good. are you going to bump shared assembly info on master and do a release?

@lgtm-com
Copy link

lgtm-com bot commented Aug 17, 2020

This pull request fixes 11 alerts when merging c313c8b into f6901d6 - view on LGTM.com

fixed alerts:

  • 4 for Useless assignment to local variable
  • 4 for Missing Dispose call on local IDisposable
  • 2 for Dereferenced variable may be null
  • 1 for Possible loss of precision

@tznind tznind merged commit 416c08b into master Aug 17, 2020
@lgtm-com
Copy link

lgtm-com bot commented Aug 17, 2020

This pull request fixes 11 alerts when merging 132a01c into f6901d6 - view on LGTM.com

fixed alerts:

  • 4 for Useless assignment to local variable
  • 4 for Missing Dispose call on local IDisposable
  • 2 for Dereferenced variable may be null
  • 1 for Possible loss of precision

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