Skip to content

fix ydb: implementation of using installed instance of ydb-cpp-sdk - #1291

Closed
vasily-sviridov wants to merge 1 commit into
userver-framework:developfrom
vasily-sviridov:sviridov-patch-ydb-install
Closed

fix ydb: implementation of using installed instance of ydb-cpp-sdk#1291
vasily-sviridov wants to merge 1 commit into
userver-framework:developfrom
vasily-sviridov:sviridov-patch-ydb-install

Conversation

@vasily-sviridov

@vasily-sviridov vasily-sviridov commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Note: by creating a PR or an issue you automatically agree to the CLA. See CONTRIBUTING.md. Feel free to remove this note, the agreement holds.

@vasily-sviridov vasily-sviridov changed the title fix(ydb): load installed package dependencies before targets fix(ydb): implementation of using installed instance of ydb Jul 8, 2026
@vasily-sviridov vasily-sviridov changed the title fix(ydb): implementation of using installed instance of ydb fix(ydb): implementation of using installed instance of ydb-cpp-sdk Jul 9, 2026
@apolukhin apolukhin changed the title fix(ydb): implementation of using installed instance of ydb-cpp-sdk fix ydb: implementation of using installed instance of ydb-cpp-sdk Aug 2, 2026
@apolukhin

Copy link
Copy Markdown
Member

LGTM

apolukhin
apolukhin previously approved these changes Aug 3, 2026
@apolukhin apolukhin closed this Aug 9, 2026
@apolukhin apolukhin reopened this Aug 9, 2026
@apolukhin
apolukhin requested a review from kpavlov00 as a code owner August 9, 2026 10:52
@apolukhin

Copy link
Copy Markdown
Member

@vasily-sviridov please rebase the PR. We're eager to merge it :)

@vasily-sviridov

Copy link
Copy Markdown
Contributor Author

@apolukhin Done! I rebased it and dropped the SetupYdbCppSDK.cmake bits since upstream handles that now.

@apolukhin

Copy link
Copy Markdown
Member

LGTM

@robot-magpie

robot-magpie Bot commented Aug 9, 2026

Copy link
Copy Markdown

Many thanks for the PR! @apolukhin is now importing your pull request into our internal upstream repository.

@robot-magpie

robot-magpie Bot commented Aug 10, 2026

Copy link
Copy Markdown

✅ This pull request is being closed because it has been successfully merged into our internal monorepository.
Your changes will be pushed to this repository soon. Thank you for your contribution!

@robot-magpie robot-magpie Bot closed this Aug 10, 2026
robot-piglet pushed a commit that referenced this pull request Aug 10, 2026
---

Pull Request resolved: #1291
commit_hash:bd8e3fd933186e56fc8984024b877b41a1bdb776
@vasily-sviridov
vasily-sviridov deleted the sviridov-patch-ydb-install branch August 10, 2026 14:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants