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.

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

Remove runtime from tests #24

Closed
wants to merge 1 commit into from
Closed

Conversation

laskoviymishka
Copy link
Contributor

We have not YT-runtime in open-source version and don't need it in e2e tests

Runtime is optional for tests, and we don't have YT-runtime in opensource version
Copy link

robot-magpie bot commented Aug 24, 2024

✅ All contributors are covered under a CLA with Yandex

See CONTRIBUTING.md for more info about Yandex Contributor License Agreement.


The following contributors were found:
dd300fb Author: @laskoviymishka (las*******shka@gmail.com)

(Only the first commit for a unique contributor is listed)

@laskoviymishka
Copy link
Contributor Author

Shipped!

Copy link

robot-magpie bot commented Aug 24, 2024

@laskoviymishka has imported your pull request. If you are a Yandex employee, you can view this diff.

@e-jaku
Copy link

e-jaku commented Aug 26, 2024

something seems up with ytsaurus lib

@laskoviymishka
Copy link
Contributor Author

laskoviymishka commented Aug 26, 2024

Yap, we need to wait till they released a new version, last one was released in june.

Copy link

robot-magpie bot commented Aug 26, 2024

✅ 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 26, 2024
robot-piglet pushed a commit that referenced this pull request Aug 26, 2024
We have not YT-runtime in open-source version and don't need it in e2e tests

---
f10e1e288d82ccff812bf0eff04b11632b7ffdaf

Pull Request resolved: #24
@laskoviymishka laskoviymishka deleted the remove-useless-runtimes branch September 14, 2024 22:50
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