Skip to content

feat: Add Flush(bool) to FileSystemStream. - #985

Merged
mergify[bot] merged 3 commits into
TestableIO:mainfrom
gregington:feat/add-flush-bool-to-filesystemstream
May 12, 2023
Merged

feat: Add Flush(bool) to FileSystemStream.#985
mergify[bot] merged 3 commits into
TestableIO:mainfrom
gregington:feat/add-flush-bool-to-filesystemstream

Conversation

@gregington

Copy link
Copy Markdown
Contributor

FileSystemStream was missing the implementation of FileStream.Flush(bool) which allows the caller to specify whether Flush should ensure all buffers are written to disk.

@vbreussvbreuss left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks a lot!
Looks good to me.

Comment threadsrc/TestableIO.System.IO.Abstractions/FileSystemStream.cs

@fgreinacherfgreinacher left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Awesome, thanks a lot!

@mergify
mergifyBot merged commit ccdc24d into TestableIO:mainMay 12, 2023
@gregington
gregington deleted the feat/add-flush-bool-to-filesystemstream branch May 12, 2023 15:08
@github-actions

Copy link
Copy Markdown

This is addressed in release v19.2.25.

@github-actionsgithub-actionsBot added the state: released Issues that are released label May 12, 2023
vbreuss referenced this pull request in Testably/Testably.Abstractions May 12, 2023
…19.2.26 (#306)
[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)
This PR contains the following updates:
| Package | Type | Update | Change |
|---|---|---|---|
|
[TestableIO.System.IO.Abstractions](https://togithub.com/TestableIO/System.IO.Abstractions)
| nuget | patch | `19.2.22` -> `19.2.26` |
---
### Release Notes
<details>
<summary>TestableIO/System.IO.Abstractions</summary>
###
[`v19.2.26`](https://togithub.com/TestableIO/System.IO.Abstractions/releases/tag/v19.2.26)
##### What's Changed
- fix: Refresh FileInfo & DirectoryInfo after adding to MockFileSystem
by [@&#8203;rcdailey](https://togithub.com/rcdailey) in
[https://github.com/TestableIO/System.IO.Abstractions/pull/984](https://togithub.com/TestableIO/System.IO.Abstractions/pull/984)
**Full Changelog**:
TestableIO/System.IO.Abstractions@v19.2.25...v19.2.26
###
[`v19.2.25`](https://togithub.com/TestableIO/System.IO.Abstractions/releases/tag/v19.2.25)
#### What's Changed
- chore(deps): update dependency githubactionstestlogger to v2.1.0 by
[@&#8203;renovate](https://togithub.com/renovate) in
[https://github.com/TestableIO/System.IO.Abstractions/pull/986](https://togithub.com/TestableIO/System.IO.Abstractions/pull/986)
- chore(deps): update dependency nerdbank.gitversioning to v3.6.132 by
[@&#8203;renovate](https://togithub.com/renovate) in
[https://github.com/TestableIO/System.IO.Abstractions/pull/987](https://togithub.com/TestableIO/System.IO.Abstractions/pull/987)
- feat: Add Flush(bool) to FileSystemStream. by
[@&#8203;gregington](https://togithub.com/gregington) in
[https://github.com/TestableIO/System.IO.Abstractions/pull/985](https://togithub.com/TestableIO/System.IO.Abstractions/pull/985)
**Full Changelog**:
TestableIO/System.IO.Abstractions@v19.2.22...v19.2.25
</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://app.renovatebot.com/dashboard#github/Testably/Testably.Abstractions).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNS43OS4xIiwidXBkYXRlZEluVmVyIjoiMzUuNzkuMSIsInRhcmdldEJyYW5jaCI6Im1haW4ifQ==-->
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
vbreuss referenced this pull request in Testably/Testably.Abstractions May 12, 2023
…19.2.26 (#306)
[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)
This PR contains the following updates:
| Package | Type | Update | Change |
|---|---|---|---|
|
[TestableIO.System.IO.Abstractions](https://togithub.com/TestableIO/System.IO.Abstractions)
| nuget | patch | `19.2.22` -> `19.2.26` |
---
### Release Notes
<details>
<summary>TestableIO/System.IO.Abstractions</summary>
###
[`v19.2.26`](https://togithub.com/TestableIO/System.IO.Abstractions/releases/tag/v19.2.26)
##### What's Changed
- fix: Refresh FileInfo & DirectoryInfo after adding to MockFileSystem
by [@&#8203;rcdailey](https://togithub.com/rcdailey) in
[https://github.com/TestableIO/System.IO.Abstractions/pull/984](https://togithub.com/TestableIO/System.IO.Abstractions/pull/984)
**Full Changelog**:
TestableIO/System.IO.Abstractions@v19.2.25...v19.2.26
###
[`v19.2.25`](https://togithub.com/TestableIO/System.IO.Abstractions/releases/tag/v19.2.25)
#### What's Changed
- chore(deps): update dependency githubactionstestlogger to v2.1.0 by
[@&#8203;renovate](https://togithub.com/renovate) in
[https://github.com/TestableIO/System.IO.Abstractions/pull/986](https://togithub.com/TestableIO/System.IO.Abstractions/pull/986)
- chore(deps): update dependency nerdbank.gitversioning to v3.6.132 by
[@&#8203;renovate](https://togithub.com/renovate) in
[https://github.com/TestableIO/System.IO.Abstractions/pull/987](https://togithub.com/TestableIO/System.IO.Abstractions/pull/987)
- feat: Add Flush(bool) to FileSystemStream. by
[@&#8203;gregington](https://togithub.com/gregington) in
[https://github.com/TestableIO/System.IO.Abstractions/pull/985](https://togithub.com/TestableIO/System.IO.Abstractions/pull/985)
**Full Changelog**:
TestableIO/System.IO.Abstractions@v19.2.22...v19.2.25
</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://app.renovatebot.com/dashboard#github/Testably/Testably.Abstractions).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNS43OS4xIiwidXBkYXRlZEluVmVyIjoiMzUuNzkuMSIsInRhcmdldEJyYW5jaCI6Im1haW4ifQ==-->
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
vbreuss added a commit to Testably/Testably.Abstractions that referenced this pull request May 12, 2023
Add support for `Flush(bool)` added in
TestableIO/System.IO.Abstractions#985
---------
Co-authored-by: Valentin Breuß <v.breuss@tig.at>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

state: releasedIssues that are released

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@gregington@fgreinacher@vbreuss