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

chore: Keep v1 and v0 in envd #1272

Merged
merged 10 commits into from
Dec 7, 2022
Merged

chore: Keep v1 and v0 in envd #1272

merged 10 commits into from
Dec 7, 2022

Conversation

gaocegege
Copy link
Member

Signed-off-by: Ce Gao [email protected]

@aseaday
Copy link
Member

aseaday commented Dec 6, 2022

A good signal to step forward to a more promising future of API. But could we have a guide for developers a whole view of v2?

@gaocegege
Copy link
Member Author

A good signal to step forward to a more promising future of API. But could we have a guide for developers a whole view of v2?

I think we should have such a guide. Maybe after #1228 is merged

/cc @kemingy

Signed-off-by: Ce Gao <[email protected]>
Signed-off-by: Ce Gao <[email protected]>
Signed-off-by: Ce Gao <[email protected]>
Signed-off-by: Ce Gao <[email protected]>
Signed-off-by: Ce Gao <[email protected]>
Signed-off-by: Ce Gao <[email protected]>
Signed-off-by: Ce Gao <[email protected]>
Signed-off-by: Ce Gao <[email protected]>
@gaocegege gaocegege changed the title WIP chore: Keep v1 and v2 in envd WIP chore: Keep v1 and v0 in envd Dec 7, 2022
@gaocegege gaocegege changed the title WIP chore: Keep v1 and v0 in envd chore: Keep v1 and v0 in envd Dec 7, 2022
@kemingy kemingy linked an issue Dec 7, 2022 that may be closed by this pull request
pkg/app/create.go Outdated Show resolved Hide resolved
pkg/lang/frontend/starlark/v0/interpreter_test.go Outdated Show resolved Hide resolved
pkg/lang/frontend/starlark/v0/starlark_suite_test.go Outdated Show resolved Hide resolved
pkg/lang/frontend/starlark/v1/testdata/test.envd Outdated Show resolved Hide resolved
Signed-off-by: Ce Gao <[email protected]>
Signed-off-by: Ce Gao <[email protected]>
@gaocegege
Copy link
Member Author

Comments are addressed, please take a look.
Thanks for your time 😄

@kemingy
Copy link
Member

kemingy commented Dec 7, 2022

CI report failed because it should needs: v1.... @gaocegege

@kemingy
Copy link
Member

kemingy commented Dec 7, 2022

Will fix the CI test coverage report in another PR.

@kemingy kemingy merged commit 09e5655 into tensorchord:main Dec 7, 2022
@VoVAllen
Copy link
Member

VoVAllen commented Dec 7, 2022

tql

@gaocegege gaocegege deleted the v1v2 branch December 7, 2022 13:09
AlexXi19 pushed a commit to AlexXi19/envd that referenced this pull request Dec 12, 2022
* feat: Add v1 and v0

Signed-off-by: Ce Gao <[email protected]>

* fix: Fix lint

Signed-off-by: Ce Gao <[email protected]>

* fix: Fix

Signed-off-by: Ce Gao <[email protected]>

* fix: Mock generation

Signed-off-by: Ce Gao <[email protected]>

* fix: Add build

Signed-off-by: Ce Gao <[email protected]>

* fix: Fix test cases

Signed-off-by: Ce Gao <[email protected]>

* fix: Update

Signed-off-by: Ce Gao <[email protected]>

* fix: Fix

Signed-off-by: Ce Gao <[email protected]>

* fix: Update

Signed-off-by: Ce Gao <[email protected]>

* fix: Fix test case

Signed-off-by: Ce Gao <[email protected]>

Signed-off-by: Ce Gao <[email protected]>
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.

discussion: How to maintain different frontend language versions
4 participants