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

Jump to bottom

Начинаем начинать github as database #138

Draft
wants to merge 57 commits into
base: master
Choose a base branch
from
Draft

Начинаем начинать github as database #138

wants to merge 57 commits into from

Conversation

zetroot
Copy link
Collaborator

@zetroot zetroot commented Apr 29, 2021 •
edited
Loading

PR для прозрачности прогресса.
Реализация описанного в #129
Набор классов для работы с репозиториями github и их содержимым в памяти. Сейчас все это реализует IFileProvider. Только стрим с файлом можно не только читать, но и писать.
Нужна собственная абстракция провайдера файлов с методами создания и удаления файлов. Обновление в репозитории осуществляется через Flush стрима.
Все начинается с GitHubFilesystem. Надо начать сессию редактирования методом StartContext, затем можно пользоваться как IFileProvider.

Что в ближайших планах:

  • увеличить покрытие тестами
  • завершение сессии редактирования - создание PR в головной репозиторий

@zetroot
cleanup old code
2aa4441
@zetroot
added sample blazor app and extracted razor components library from it
3871b7d
@zetroot
drop previous attempts
a991721
@zetroot
new template project
bbb391f
@zetroot
wip
c194341
@zetroot
github client wip
140acc2
@zetroot
finally got how to create pr between forks
526159b
@zetroot
implementing ifilesystem
7f2372f
@zetroot
renaming to commune
ca6c96e
@zetroot
building github filesystem
fd5a694
@zetroot
wip
c0b948f
@zetroot
test wip
131d1e3
@zetroot
tests wip
0891954
@zetroot
rebuild stream creation
9dab8a0
@zetroot
removed setting data in stream constructor
637bc56
@zetroot
me failed with git
e93fa65
@zetroot
GithubFile tests implementation
fa1e982
@zetroot
removed template project to reduce PR size
9170b3a
@zetroot
DirectoryContents tests implementation
0c8697b
@zetroot
Merge remote-tracking branch 'origin/serverles-poc' into serverless-poc
f7d672f 
# Conflicts:
#	DotNetRu.Commune.GitHubFilesystem/DotNetRu.Commune.GitHubFilesystem.csproj
#	DotNetRu.Commune.GitHubFilesystem/EditingContext.cs
#	DotNetRu.Commune.GitHubFilesystem/GitHubFileStream.cs
#	DotNetRu.Commune.GitHubFilesystem/GitHubFilesystem.cs
#	DotNetRu.Commune.sln
#	test/DotNetRu.Commune.Test.Fs.GitHubFilesystem/DotNetRu.Commune.Test.Fs.GitHubFilesystem.fsproj
#	test/DotNetRu.Commune.Test.Fs.GitHubFilesystem/GitHubFileStreamTests.fs
@zetroot
classes visibility tuning, docs added, added specific exception type
eaeae5d
@zetroot
added commit implementation to context
69c7cb5
@zetroot
added commit method to GitHubFilesystem, added tests for it
a8cb923
@zetroot
Copy link
Collaborator Author

zetroot commented May 3, 2021

Добавил возможность закоммитить изменения.
В тестах лежит файл для F# Interactive , где можно поиграться с клиентом.
test/DotNetRu.Commune.Test.Fs.GitHubFilesystem/GithubScriptTest.fsx
Чтобы все заработало его надо покормить токеном авторизации (первый аргумент метода StartContext)
В результате можно получить вот такой прекрасный PR

@kulakovt
Copy link
Member

kulakovt commented May 3, 2021

@zetroot Было бы замечательно получить подробную инструкцию как получить GitHub токен. И сохранить её в wiki

@zetroot
Copy link
Collaborator Author

zetroot commented May 3, 2021

@kulakovt ок, займусь завтра.

@zetroot
Copy link
Collaborator Author

zetroot commented May 4, 2021

@kulakovt добавил страницу в wiki

@zetroot
Logging and configuration (#2)
4d36765 
* setting up serilog logging

* dependency injection and options injection
@zetroot
Merge remote-tracking branch 'upstream/master'
c464cd9
@zetroot
updated from upstream
26b1b73
zetroot added 24 commits May 8, 2021 17:51
@zetroot
github client wip
65c2360
@zetroot
finally got how to create pr between forks
356c1e5
@zetroot
implementing ifilesystem
08becf8
@zetroot
renaming to commune
adf57f1
@zetroot
building github filesystem
e01722e
@zetroot
wip
f94d740
@zetroot
test wip
759c906
@zetroot
tests wip
d279514
@zetroot
rebuild stream creation
f7dcb5e
@zetroot
removed setting data in stream constructor
d6922ef
@zetroot
me failed with git
c418cfb
@zetroot
GithubFile tests implementation
89dba2a
@zetroot
removed template project to reduce PR size
802504b
@zetroot
DirectoryContents tests implementation
439aaf2
@zetroot
classes visibility tuning, docs added, added specific exception type
f443f67
@zetroot
added commit implementation to context
f0b8d8a
@zetroot
added commit method to GitHubFilesystem, added tests for it
d3b1285
@zetroot
Merge remote-tracking branch 'origin/serverles-poc' into serverless-poc
3ccd3ed
@zetroot
fixed solution file
4d1f74a
@zetroot
first hello world app wip
3a5f8e2
@zetroot
directory viewer rebuilt
58e49b9
@zetroot
file contents viewer
cf85578
@zetroot
trying to change files
7289b0a
@zetroot
reading and writing file
2a2cbbf
@zetroot
Copy link
Collaborator Author

zetroot commented Jun 1, 2021

Коллеги, что-то я застрял.

  • синхронные API нам не подойдут - blazor не может в Task.Result или в .GetAwaiter().GetResult(), поэтому вся работа с "файлами" должна быть асинхронной
  • сейчас приложуха умеет дописывать в открытый файл и делать PR, но вот тут мне не хватает идей. Что нам еще нужно для PoC?
  • я вспомнил почему я не пошел в веб

@kulakovt
Copy link
Member

kulakovt commented Jun 1, 2021

вся работа с "файлами" должна быть асинхронной

Думаю, это не проблема. Любой IO должен быть асинхронным.

Что нам еще нужно для PoC?

Игорь писал в #129

  • Чтение файлов
  • Создание/удаление бранчей
  • Создание PR
  • Я бы ещё добавил создание/запись в файл

@zetroot
Copy link
Collaborator Author

zetroot commented Jun 2, 2021

Итак, я немного потрудился еще и теперь:

  • можно создавать бранчи (создается автоматически при начале редактирования)
  • можно читать файлы!
  • можно создавать PR в основной репозиторий
  • можно создавать файлы (тут есть ограничение - нельзя создать пустой файл, там обязательно что-то должно быть, поэтому сначала файл создаётся виртуально, а при Flush будет запрос на Create а не Update)

Интерфейс конечно выглядит отвратительно. Но вроде бы работает. Кто-то может испытать?

@kulakovt
Copy link
Member

Посмотрите пожалуйста асинхронную файловую систему. Подходит ли она под все ваши пожелания?
https://github.com/DotNetRu/Auditor/pull/8/files

@zetroot
Copy link
Collaborator Author

zetroot commented Jun 11, 2021

@kulakovt выглядит в целом как то что нужно. Хочу еще больше асинхронности, например в получении Stream. Отписал в PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
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
@zetroot @kulakovt