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

fix: nullable warning from SonarCloud #400

Merged
merged 2 commits into from
Sep 10, 2023
Merged

Conversation

vbreuss
Copy link
Member

@vbreuss vbreuss commented Sep 10, 2023

Fix incorrect nullable warning from SonarCloud:
image

@vbreuss vbreuss added the bug Something isn't working label Sep 10, 2023
@vbreuss vbreuss self-assigned this Sep 10, 2023
@vbreuss vbreuss enabled auto-merge (squash) September 10, 2023 13:15
@vbreuss vbreuss merged commit 22f13f4 into main Sep 10, 2023
7 checks passed
@vbreuss vbreuss deleted the topic/vb/fix-sonar-warning branch September 10, 2023 13:24
mergify bot pushed a commit that referenced this pull request Oct 7, 2023
[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
|
[Testably.Abstractions](https://github.com/Testably/Testably.Abstractions)
| `2.4.1` -> `2.5.0` |
[![age](https://developer.mend.io/api/mc/badges/age/nuget/Testably.Abstractions/2.5.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/nuget/Testably.Abstractions/2.5.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/nuget/Testably.Abstractions/2.4.1/2.5.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/nuget/Testably.Abstractions/2.4.1/2.5.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
|
[Testably.Abstractions.Testing](https://github.com/Testably/Testably.Abstractions)
| `2.4.1` -> `2.5.0` |
[![age](https://developer.mend.io/api/mc/badges/age/nuget/Testably.Abstractions.Testing/2.5.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/nuget/Testably.Abstractions.Testing/2.5.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/nuget/Testably.Abstractions.Testing/2.4.1/2.5.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/nuget/Testably.Abstractions.Testing/2.4.1/2.5.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

### Release Notes

<details>
<summary>Testably/Testably.Abstractions
(Testably.Abstractions)</summary>

###
[`v2.5.0`](https://github.com/Testably/Testably.Abstractions/releases/tag/v2.5.0)

##### What's Changed

- chore(deps): update testably.abstractions packages to v2.4.1 by
[@&#8203;renovate](https://github.com/renovate) in
[#379
- refactor: prepare support for .NET 8.0 by
[@&#8203;vbreuss](https://github.com/vbreuss) in
[#388
- feat: support .NET 8.0 | `LastAccessTime` is no longer updated for the
source file when copying it on OSX by
[@&#8203;vbreuss](https://github.com/vbreuss) in
[#382
- feat: support .NET 8.0 | Remove `BinaryFormatter` from
`TestingException` by [@&#8203;vbreuss](https://github.com/vbreuss) in
[#383
- feat: support .NET 8.0 | Refresh cache on `IFileInfo.CreateText` by
[@&#8203;vbreuss](https://github.com/vbreuss) in
[#384
- feat: support .NET 8.0 | Timer no longer throws
`ObjectDisposedException` by
[@&#8203;vbreuss](https://github.com/vbreuss) in
[#385
- feat: support .NET 8.0 | Add new methods for `IZipFile` by
[@&#8203;vbreuss](https://github.com/vbreuss) in
[#386
- feat: support .NET 8.0 | Add new methods for `IRandom` by
[@&#8203;vbreuss](https://github.com/vbreuss) in
[#387
- chore(deps): update dependency testableio.system.io.abstractions to
v19.2.67 by [@&#8203;renovate](https://github.com/renovate) in
[#390
- chore(deps): update dependency microsoft.net.test.sdk to v17.7.2 by
[@&#8203;renovate](https://github.com/renovate) in
[#391
- chore(deps): update dependency testableio.system.io.abstractions to
v19.2.69 by [@&#8203;renovate](https://github.com/renovate) in
[#392
- chore(deps): update dependency testably.abstractions.fluentassertions
to v0.7.0 by [@&#8203;renovate](https://github.com/renovate) in
[#393
- chore(deps): update actions/checkout action to v4 by
[@&#8203;renovate](https://github.com/renovate) in
[#394
- chore(deps): update danielpalme/reportgenerator-github-action action
to v5.1.25 by [@&#8203;renovate](https://github.com/renovate) in
[#395
- chore(deps): update dependency testably.abstractions.fluentassertions
to v0.8.0 by [@&#8203;renovate](https://github.com/renovate) in
[#396
- chore(deps): update dependency testably.abstractions.fluentassertions
to v0.9.0 by [@&#8203;renovate](https://github.com/renovate) in
[#397
- refactor: use `Exist` and `NotExist` fluent assertions by
[@&#8203;vbreuss](https://github.com/vbreuss) in
[#398
- fix: nullable warning from SonarCloud by
[@&#8203;vbreuss](https://github.com/vbreuss) in
[#400
- chore(deps): update dependency nsubstitute to v5.1.0 by
[@&#8203;renovate](https://github.com/renovate) in
[#401
- chore(deps): update xunit-dotnet monorepo to v2.5.1 by
[@&#8203;renovate](https://github.com/renovate) in
[#402
- chore(deps): update actions/checkout digest to
[`8ade135`](https://github.com/Testably/Testably.Abstractions/commit/8ade135)
by [@&#8203;renovate](https://github.com/renovate) in
[#404
- docs: update README by [@&#8203;vbreuss](https://github.com/vbreuss)
in
[#405
- chore(deps): update danielpalme/reportgenerator-github-action action
to v5.1.26 by [@&#8203;renovate](https://github.com/renovate) in
[#407
- fix: explicitly specify `FileAttributes` in tests by
[@&#8203;vbreuss](https://github.com/vbreuss) in
[#406
- fix: throw UnauthorizedAccessException when replacing a read-only file
on Windows by [@&#8203;vbreuss](https://github.com/vbreuss) in
[#408
- chore(deps): update dependency testably.abstractions.fluentassertions
to v1 by [@&#8203;renovate](https://github.com/renovate) in
[#409

**Full Changelog**:
v2.4.1...v2.5.0

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about these
updates again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/Testably/Testably.Abstractions).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4wLjMiLCJ1cGRhdGVkSW5WZXIiOiIzNy4wLjMiLCJ0YXJnZXRCcmFuY2giOiJtYWluIn0=-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
mergify bot pushed a commit to Testably/Testably.Abstractions.FluentAssertions that referenced this pull request Oct 7, 2023
#56)

[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
|
[Testably.Abstractions.Testing](https://github.com/Testably/Testably.Abstractions)
| `2.4.1` -> `2.5.0` |
[![age](https://developer.mend.io/api/mc/badges/age/nuget/Testably.Abstractions.Testing/2.5.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/nuget/Testably.Abstractions.Testing/2.5.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/nuget/Testably.Abstractions.Testing/2.4.1/2.5.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/nuget/Testably.Abstractions.Testing/2.4.1/2.5.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

### Release Notes

<details>
<summary>Testably/Testably.Abstractions
(Testably.Abstractions.Testing)</summary>

###
[`v2.5.0`](https://github.com/Testably/Testably.Abstractions/releases/tag/v2.5.0)

##### What's Changed

- chore(deps): update testably.abstractions packages to v2.4.1 by
[@&#8203;renovate](https://github.com/renovate) in
[Testably/Testably.Abstractions#379
- refactor: prepare support for .NET 8.0 by
[@&#8203;vbreuss](https://github.com/vbreuss) in
[Testably/Testably.Abstractions#388
- feat: support .NET 8.0 | `LastAccessTime` is no longer updated for the
source file when copying it on OSX by
[@&#8203;vbreuss](https://github.com/vbreuss) in
[Testably/Testably.Abstractions#382
- feat: support .NET 8.0 | Remove `BinaryFormatter` from
`TestingException` by [@&#8203;vbreuss](https://github.com/vbreuss) in
[Testably/Testably.Abstractions#383
- feat: support .NET 8.0 | Refresh cache on `IFileInfo.CreateText` by
[@&#8203;vbreuss](https://github.com/vbreuss) in
[Testably/Testably.Abstractions#384
- feat: support .NET 8.0 | Timer no longer throws
`ObjectDisposedException` by
[@&#8203;vbreuss](https://github.com/vbreuss) in
[Testably/Testably.Abstractions#385
- feat: support .NET 8.0 | Add new methods for `IZipFile` by
[@&#8203;vbreuss](https://github.com/vbreuss) in
[Testably/Testably.Abstractions#386
- feat: support .NET 8.0 | Add new methods for `IRandom` by
[@&#8203;vbreuss](https://github.com/vbreuss) in
[Testably/Testably.Abstractions#387
- chore(deps): update dependency testableio.system.io.abstractions to
v19.2.67 by [@&#8203;renovate](https://github.com/renovate) in
[Testably/Testably.Abstractions#390
- chore(deps): update dependency microsoft.net.test.sdk to v17.7.2 by
[@&#8203;renovate](https://github.com/renovate) in
[Testably/Testably.Abstractions#391
- chore(deps): update dependency testableio.system.io.abstractions to
v19.2.69 by [@&#8203;renovate](https://github.com/renovate) in
[Testably/Testably.Abstractions#392
- chore(deps): update dependency testably.abstractions.fluentassertions
to v0.7.0 by [@&#8203;renovate](https://github.com/renovate) in
[Testably/Testably.Abstractions#393
- chore(deps): update actions/checkout action to v4 by
[@&#8203;renovate](https://github.com/renovate) in
[Testably/Testably.Abstractions#394
- chore(deps): update danielpalme/reportgenerator-github-action action
to v5.1.25 by [@&#8203;renovate](https://github.com/renovate) in
[Testably/Testably.Abstractions#395
- chore(deps): update dependency testably.abstractions.fluentassertions
to v0.8.0 by [@&#8203;renovate](https://github.com/renovate) in
[Testably/Testably.Abstractions#396
- chore(deps): update dependency testably.abstractions.fluentassertions
to v0.9.0 by [@&#8203;renovate](https://github.com/renovate) in
[Testably/Testably.Abstractions#397
- refactor: use `Exist` and `NotExist` fluent assertions by
[@&#8203;vbreuss](https://github.com/vbreuss) in
[Testably/Testably.Abstractions#398
- fix: nullable warning from SonarCloud by
[@&#8203;vbreuss](https://github.com/vbreuss) in
[Testably/Testably.Abstractions#400
- chore(deps): update dependency nsubstitute to v5.1.0 by
[@&#8203;renovate](https://github.com/renovate) in
[Testably/Testably.Abstractions#401
- chore(deps): update xunit-dotnet monorepo to v2.5.1 by
[@&#8203;renovate](https://github.com/renovate) in
[Testably/Testably.Abstractions#402
- chore(deps): update actions/checkout digest to
[`8ade135`](https://github.com/Testably/Testably.Abstractions/commit/8ade135)
by [@&#8203;renovate](https://github.com/renovate) in
[Testably/Testably.Abstractions#404
- docs: update README by [@&#8203;vbreuss](https://github.com/vbreuss)
in
[Testably/Testably.Abstractions#405
- chore(deps): update danielpalme/reportgenerator-github-action action
to v5.1.26 by [@&#8203;renovate](https://github.com/renovate) in
[Testably/Testably.Abstractions#407
- fix: explicitly specify `FileAttributes` in tests by
[@&#8203;vbreuss](https://github.com/vbreuss) in
[Testably/Testably.Abstractions#406
- fix: throw UnauthorizedAccessException when replacing a read-only file
on Windows by [@&#8203;vbreuss](https://github.com/vbreuss) in
[Testably/Testably.Abstractions#408
- chore(deps): update dependency testably.abstractions.fluentassertions
to v1 by [@&#8203;renovate](https://github.com/renovate) in
[Testably/Testably.Abstractions#409

**Full Changelog**:
Testably/Testably.Abstractions@v2.4.1...v2.5.0

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/Testably/Testably.Abstractions.FluentAssertions).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4wLjMiLCJ1cGRhdGVkSW5WZXIiOiIzNy4wLjMiLCJ0YXJnZXRCcmFuY2giOiJtYWluIn0=-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@github-actions
Copy link

github-actions bot commented Oct 7, 2023

This is addressed in release v2.5.0.

vbreuss pushed a commit to Testably/Testably.Abstractions.FluentAssertions that referenced this pull request Oct 7, 2023
…5.0 (#55)

[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
|
[Testably.Abstractions.Interface](https://github.com/Testably/Testably.Abstractions)
| `2.4.1` -> `2.5.0` |
[![age](https://developer.mend.io/api/mc/badges/age/nuget/Testably.Abstractions.Interface/2.5.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/nuget/Testably.Abstractions.Interface/2.5.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/nuget/Testably.Abstractions.Interface/2.4.1/2.5.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/nuget/Testably.Abstractions.Interface/2.4.1/2.5.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

### Release Notes

<details>
<summary>Testably/Testably.Abstractions
(Testably.Abstractions.Interface)</summary>

###
[`v2.5.0`](https://github.com/Testably/Testably.Abstractions/releases/tag/v2.5.0)

##### What's Changed

- chore(deps): update testably.abstractions packages to v2.4.1 by
[@&#8203;renovate](https://github.com/renovate) in
[Testably/Testably.Abstractions#379
- refactor: prepare support for .NET 8.0 by
[@&#8203;vbreuss](https://github.com/vbreuss) in
[Testably/Testably.Abstractions#388
- feat: support .NET 8.0 | `LastAccessTime` is no longer updated for the
source file when copying it on OSX by
[@&#8203;vbreuss](https://github.com/vbreuss) in
[Testably/Testably.Abstractions#382
- feat: support .NET 8.0 | Remove `BinaryFormatter` from
`TestingException` by [@&#8203;vbreuss](https://github.com/vbreuss) in
[Testably/Testably.Abstractions#383
- feat: support .NET 8.0 | Refresh cache on `IFileInfo.CreateText` by
[@&#8203;vbreuss](https://github.com/vbreuss) in
[Testably/Testably.Abstractions#384
- feat: support .NET 8.0 | Timer no longer throws
`ObjectDisposedException` by
[@&#8203;vbreuss](https://github.com/vbreuss) in
[Testably/Testably.Abstractions#385
- feat: support .NET 8.0 | Add new methods for `IZipFile` by
[@&#8203;vbreuss](https://github.com/vbreuss) in
[Testably/Testably.Abstractions#386
- feat: support .NET 8.0 | Add new methods for `IRandom` by
[@&#8203;vbreuss](https://github.com/vbreuss) in
[Testably/Testably.Abstractions#387
- chore(deps): update dependency testableio.system.io.abstractions to
v19.2.67 by [@&#8203;renovate](https://github.com/renovate) in
[Testably/Testably.Abstractions#390
- chore(deps): update dependency microsoft.net.test.sdk to v17.7.2 by
[@&#8203;renovate](https://github.com/renovate) in
[Testably/Testably.Abstractions#391
- chore(deps): update dependency testableio.system.io.abstractions to
v19.2.69 by [@&#8203;renovate](https://github.com/renovate) in
[Testably/Testably.Abstractions#392
- chore(deps): update dependency testably.abstractions.fluentassertions
to v0.7.0 by [@&#8203;renovate](https://github.com/renovate) in
[Testably/Testably.Abstractions#393
- chore(deps): update actions/checkout action to v4 by
[@&#8203;renovate](https://github.com/renovate) in
[Testably/Testably.Abstractions#394
- chore(deps): update danielpalme/reportgenerator-github-action action
to v5.1.25 by [@&#8203;renovate](https://github.com/renovate) in
[Testably/Testably.Abstractions#395
- chore(deps): update dependency testably.abstractions.fluentassertions
to v0.8.0 by [@&#8203;renovate](https://github.com/renovate) in
[Testably/Testably.Abstractions#396
- chore(deps): update dependency testably.abstractions.fluentassertions
to v0.9.0 by [@&#8203;renovate](https://github.com/renovate) in
[Testably/Testably.Abstractions#397
- refactor: use `Exist` and `NotExist` fluent assertions by
[@&#8203;vbreuss](https://github.com/vbreuss) in
[Testably/Testably.Abstractions#398
- fix: nullable warning from SonarCloud by
[@&#8203;vbreuss](https://github.com/vbreuss) in
[Testably/Testably.Abstractions#400
- chore(deps): update dependency nsubstitute to v5.1.0 by
[@&#8203;renovate](https://github.com/renovate) in
[Testably/Testably.Abstractions#401
- chore(deps): update xunit-dotnet monorepo to v2.5.1 by
[@&#8203;renovate](https://github.com/renovate) in
[Testably/Testably.Abstractions#402
- chore(deps): update actions/checkout digest to
[`8ade135`](https://github.com/Testably/Testably.Abstractions/commit/8ade135)
by [@&#8203;renovate](https://github.com/renovate) in
[Testably/Testably.Abstractions#404
- docs: update README by [@&#8203;vbreuss](https://github.com/vbreuss)
in
[Testably/Testably.Abstractions#405
- chore(deps): update danielpalme/reportgenerator-github-action action
to v5.1.26 by [@&#8203;renovate](https://github.com/renovate) in
[Testably/Testably.Abstractions#407
- fix: explicitly specify `FileAttributes` in tests by
[@&#8203;vbreuss](https://github.com/vbreuss) in
[Testably/Testably.Abstractions#406
- fix: throw UnauthorizedAccessException when replacing a read-only file
on Windows by [@&#8203;vbreuss](https://github.com/vbreuss) in
[Testably/Testably.Abstractions#408
- chore(deps): update dependency testably.abstractions.fluentassertions
to v1 by [@&#8203;renovate](https://github.com/renovate) in
[Testably/Testably.Abstractions#409

**Full Changelog**:
Testably/Testably.Abstractions@v2.4.1...v2.5.0

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/Testably/Testably.Abstractions.FluentAssertions).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4wLjMiLCJ1cGRhdGVkSW5WZXIiOiIzNy4wLjMiLCJ0YXJnZXRCcmFuY2giOiJtYWluIn0=-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working state: released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant