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

tests(sdk): Fix & roll-forward "Run tests on toolbox dev version (#19)"" #90

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

anubhav756
Copy link
Contributor

@anubhav756 anubhav756 commented Mar 13, 2025

This PR rolls-forward #19.

We found issues in that PR here, so we reverted in #89.

We will fix the issue in this PR.

@anubhav756 anubhav756 requested a review from twishabansal March 13, 2025 09:23
@anubhav756 anubhav756 requested a review from a team as a code owner March 13, 2025 09:23
@twishabansal
Copy link
Contributor

Why are we rolling this forward?

@anubhav756
Copy link
Contributor Author

Why are we rolling this forward?

Not rolling forward as-is (updated the description above).

I opened it so we can track the changes that were supposed to be done in the original #19, but fix it here before merging.

Does that make sense?

@anubhav756 anubhav756 marked this pull request as draft March 13, 2025 09:30
@anubhav756 anubhav756 changed the title Roll-forward "tests(sdk): Run tests on toolbox dev version (#19)"" Fix & roll-forward "tests(sdk): Run tests on toolbox dev version (#19)"" Mar 13, 2025
@anubhav756 anubhav756 changed the title Fix & roll-forward "tests(sdk): Run tests on toolbox dev version (#19)"" tests(sdk): Fix & roll-forward "Run tests on toolbox dev version (#19)"" Mar 13, 2025
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