Skip to content

Aync File System - #8

Merged
kulakovt merged 4 commits into
masterfrom
feature/fs
Jun 11, 2021
Merged

Aync File System#8
kulakovt merged 4 commits into
masterfrom
feature/fs

Conversation

@kulakovt

@kulakovtkulakovt commented Jun 10, 2021

Copy link
Copy Markdown
Member

No description provided.

@kulakovt

kulakovt commented Jun 10, 2021

Copy link
Copy Markdown
MemberAuthor

@zetroot

zetroot
zetroot suggested changes Jun 11, 2021
View reviewed changes

@zetrootzetroot 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.

Как обсуждали - добавил ValueTask<> в получение Stream для чтения. Что ты думаешь на этот счёт?

Comment threadsrc/Storage/FileSystem/IFile.cs Outdated
Comment threadsrc/Storage/FileSystem/IFile.cs Outdated
Comment threadsrc/Storage/FileSystem/NotFoundFile.cs Outdated
Comment threadsrc/Storage/FileSystem/NotFoundFile.cs Outdated
Comment threadsrc/Storage/FileSystem/IFile.cs
Comment threadsrc/Storage/FileSystem/Physical/PhysicalFile.cs Outdated
Comment threadsrc/Storage/FileSystem/Physical/PhysicalFile.cs Outdated
Comment threadsrc/Storage/FileSystem/Physical/PhysicalFile.cs Outdated
Comment threadsrc/test/Tests/Storage/FileSystem/FileSystemEntryTest.cs Outdated
Comment threadsrc/test/Tests/Storage/FileSystem/FileSystemEntryTest.cs Outdated
@kulakovt

kulakovt commented Jun 11, 2021

Copy link
Copy Markdown
MemberAuthor

ValueTask<> в получение Stream

Сделал

@kulakovt
kulakovt merged commit 92842f2 into masterJun 11, 2021
@kulakovt
kulakovt deleted the feature/fs branch June 11, 2021 21:24
@zetroot

zetroot commented Jun 11, 2021

Copy link
Copy Markdown
Contributor

🎉

@kulakovtkulakovt mentioned this pull request Jul 31, 2021
Open
9 tasks
@kulakovtkulakovt linked an issue Jul 31, 2021 that may be closed by this pull request
Провайдеры доступа к данным #7
Open
9 tasks
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer
@zetrootzetrootzetroot requested changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Провайдеры доступа к данным

2 participants

@kulakovt@zetroot