-
Notifications
You must be signed in to change notification settings - Fork 734
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>
|
Hey @blinkov, @anton-bobkov, it has been 1216 business-hours since the author's last update, could you please review? |
|
Hey @blinkov, @anton-bobkov, it has been 1246 business-hours since the author's last update, could you please review? |
|
Hey @blinkov, @anton-bobkov, it has been 1270 business-hours since the author's last update, could you please review? |
|
Hey @blinkov, @anton-bobkov, it has been 1300 business-hours since the author's last update, could you please review? |
|
Hey @blinkov, @anton-bobkov, it has been 1330 business-hours since the author's last update, could you please review? |
|
Hey @blinkov, @anton-bobkov, it has been 1360 business-hours since the author's last update, could you please review? |
|
Hey @blinkov, @anton-bobkov, it has been 1390 business-hours since the author's last update, could you please review? |
|
Hey @blinkov, @anton-bobkov, it has been 1420 business-hours since the author's last update, could you please review? |
|
Hey @blinkov, @anton-bobkov, it has been 1444 business-hours since the author's last update, could you please review? |
|
Hey @blinkov, @anton-bobkov, it has been 1474 business-hours since the author's last update, could you please review? |
|
Hey @blinkov, @anton-bobkov, it has been 1504 business-hours since the author's last update, could you please review? |
|
Hey @blinkov, @anton-bobkov, it has been 1534 business-hours since the author's last update, could you please review? |
|
Hey @blinkov, @anton-bobkov, it has been 1564 business-hours since the author's last update, could you please review? |
|
Hey @blinkov, @anton-bobkov, it has been 1588 business-hours since the author's last update, could you please review? |
|
Hey @blinkov, @anton-bobkov, it has been 1612 business-hours since the author's last update, could you please review? |
|
Hey @orange13, it has been 1642 business-hours since the author's last update, could you please review? |
|
Hey @orange13, it has been 1672 business-hours since the author's last update, could you please review? |
|
Hey @orange13, it has been 1702 business-hours since the author's last update, could you please review? |
|
Hey @orange13, it has been 1732 business-hours since the author's last update, could you please review? |
|
Hey @orange13, it has been 1762 business-hours since the author's last update, could you please review? |
|
Hey @orange13, it has been 1792 business-hours since the author's last update, could you please review? |
|
Hey @orange13, it has been 1822 business-hours since the author's last update, could you please review? |
|
Hey @orange13, it has been 1852 business-hours since the author's last update, could you please review? |
|
Hey @orange13, it has been 1882 business-hours since the author's last update, could you please review? |
|
Hey @orange13, it has been 1912 business-hours since the author's last update, could you please review? |
|
Hey @orange13, it has been 1942 business-hours since the author's last update, could you please review? |
|
Hey @orange13, it has been 1972 business-hours since the author's last update, could you please review? |
Changelog category