Пакет для Go #9
Merged
Пакет для Go #9
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
… Add annotations to all protos
…proto generation pipeline. - Removed `go.mod` and associated setup from `finam-trade-api-go`. - Introduced `syncGoToRoot` and `generateGoPackage` Gradle tasks for managing Go proto generation. - Updated Go workflows to reflect new structure and Go version (1.24.x). - Adjusted `go_package` paths in proto files for compatibility. - Added detailed Go client `README.md` with usage instructions.
| @@ -0,0 +1,67 @@ | |||
| # Клиент к Finam Trade API для Go | |||
Collaborator
There was a problem hiding this comment.
Я не очень понимаю, зачем тут все файлы pb.go. Они ведь собираются с помощью команды build. И я так понимаю устанавливаются с go get github.com/FinamWeb/finam-trade-api/go@latest
Member
Author
There was a problem hiding this comment.
Пакет в Go это git репозиторий в котором лежат файлы.
Можно оставить так, либо воспользоваться альтернативным вариантом в описании PR
Collaborator
|
@ViktoriiaLebedeva а с версиями у нас нет конфликтов? |
Closed
# Conflicts: # proto/grpc/tradeapi/v1/assets/assets_service.proto
vishaloures
approved these changes
Dec 9, 2025
ruslansennov
approved these changes
Dec 10, 2025
aparfenov-ext
approved these changes
Dec 10, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Обзор
В данном пулл-реквесте проведен рефакторинг структуры Go-модуля и реализованы улучшения для оптимизации процессов сборки и генерации proto-файлов для Go-клиента.
В предложенном изменении пакет храниться в текущем репозитории, в дальнейшем при обновлении
protoфайлов, в вете main go пакет и спецификация OpenAPI обновиться автоматически с помощью CI.Изменения
go_packageв proto-файлах для обеспечения совместимости.README.mdдля Go-клиента с инструкциями по использованию.Важно!
Данное изменение добавляет требование версионирование в данном репозитории. Следует перейти на семантическое версионирование 1.9.0 и использовать тэги в git v1.9.0 (сейчас Release-2.8.0)
Использование пакета
Команда установки последней версии: