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.

Sign up for GitHub

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

Tidy attacher syntax/logic a bit #436

Merged
merged 2 commits into from
Sep 26, 2024

Merge branch 'develop' into feature/attachertidy

d44cf76
Select commit
Loading
Failed to load commit list.
Merged

Tidy attacher syntax/logic a bit #436

Merge branch 'develop' into feature/attachertidy
d44cf76
Select commit
Loading
Failed to load commit list.
GitHub Advanced Security / CodeQL succeeded Sep 25, 2024 in 5s

1 new alert

New alerts in code changed by this pull request

  • 1 note

See annotations below for details.

View all branch alerts.

Annotations

Check notice on line 85 in Rdmp.Dicom/Attachers/Routing/AutoRoutingAttacher.cs

See this annotation in the file changed.

Code scanning / CodeQL

Unnecessarily complex Boolean expression Note

The expression 'A == false' can be simplified to '!A'.

Check notice on line 177 in Rdmp.Dicom/Attachers/Routing/AutoRoutingAttacher.cs

See this annotation in the file changed.

Code scanning / CodeQL

Generic catch clause Note

Generic catch clause.