-
Notifications
You must be signed in to change notification settings - Fork 709
Add documentation on using Dev Container for contributors to YDB SDKs #20039
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
base: main
Are you sure you want to change the base?
Conversation
🟢 |
⚪ Test history | Ya make output | Test bloat
🟢 |
⚪ Test history | Ya make output | Test bloat
🟢 |
✅ Documentation buildRevision built successfully |
Hey @ElenaAfina, it has been 26 business-hours since the author's last update, could you please review? |
Hey @ElenaAfina, it has been 56 business-hours since the author's last update, could you please review? |
🔄 New commits pushed — @ElenaAfina please take a look. |
⚪ Test history | Ya make output | Test bloat
🟢 |
⚪ Test history | Ya make output | Test bloat
🟢 |
🔄 New commits pushed — @ElenaAfina please take a look. |
⚪ Test history | Ya make output | Test bloat
🟢 |
⚪ Test history | Ya make output | Test bloat
🟢 |
✅ Documentation buildRevision built successfully Build logsWarnings (16)
Log was truncated. (1 records) |
### Java SDK | ||
- В [`devcontainer`](https://github.com/ydb-platform/ydb-java-sdk/tree/master/.devcontainer) есть `Dockerfile`, `devcontainer.json` и скрипты. | ||
- Окружение включает JDK, Gradle и инструменты для разработки. | ||
- Можно расширять Dockerfile для установки дополнительных средств. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
вот и впиши там в скобочках: дополнительных средств (например, <ссылка на yc cli>, <можно еще пример>)
|
||
### Python SDK (PR #590) | ||
- В папке [`.devcontainer`](https://github.com/ydb-platform/ydb-python-sdk/pull/590/files) есть `Dockerfile`, `devcontainer.json`, `compose.yml` и скрипты для автоматической настройки окружения. | ||
- Все зависимости, flake8, tox уже установлены. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ссылки, что это flake8, tox
🔄 New commits pushed — @ElenaAfina please take a look. |
⚪ Test history | Ya make output | Test bloat
🟢 |
⚪ Test history | Ya make output | Test bloat
🟢 |
✅ Documentation buildRevision built successfully Build logsWarnings (16)
Log was truncated. (1 records) |
Heads-up: it's been 8 business-days since a reviewer comment. @polRk, any updates? @ElenaAfina, please check the status with the author. |
Heads-up: it's been 16 business-days since a reviewer comment. @polRk, any updates? @ElenaAfina, please check the status with the author. |
…nt SDKs Signed-off-by: Vladislav Polyakov <polRk@ydb.tech>
Signed-off-by: Vladislav Polyakov <polRk@ydb.tech>
…older for SDK references Signed-off-by: Vladislav Polyakov <polRk@ydb.tech>
Signed-off-by: Vladislav Polyakov <polRk@ydb.tech>
Co-authored-by: ElenaAfina <144937430+ElenaAfina@users.noreply.github.com>
f9cf50a
to
07faa1e
Compare
🔄 New commits pushed — @ElenaAfina please take a look. |
…ndencies are pre-installed for each SDK Signed-off-by: Vladislav Polyakov <polRk@ydb.tech>
🔄 New commits pushed — @ElenaAfina please take a look. |
⚪ Test history | Ya make output | Test bloat
🟢 |
⚪ Test history | Ya make output | Test bloat
🟢 |
✅ Documentation buildRevision built successfully Build logsWarnings (24)
Log was truncated. (9 records) |
Heads-up: it's been 24 business-days since a reviewer comment. @polRk, any updates? @ElenaAfina, please check the status with the author. |
Heads-up: it's been 32 business-days since a reviewer comment. @polRk, any updates? @ElenaAfina, please check the status with the author. |
Hey @blinkov, @anton-bobkov, it has been 790 business-hours since the author's last update, could you please review? |
Hey @blinkov, @anton-bobkov, it has been 814 business-hours since the author's last update, could you please review? |
Changelog category