Skip to content

feat(mdz deploy): Make watch by default (Updated PR) - #136

Closed
tddschn wants to merge 38 commits into
tensorchord:55-mdz-deploy-watch-by-defaultfrom
tddschn:55-mdz-deploy-watch-by-default
Closed

feat(mdz deploy): Make watch by default (Updated PR)#136
tddschn wants to merge 38 commits into
tensorchord:55-mdz-deploy-watch-by-defaultfrom
tddschn:55-mdz-deploy-watch-by-default

Conversation

@tddschn

@tddschn tddschn commented Aug 9, 2023

Copy link
Copy Markdown
Contributor

Hi @gaocegege , I've been learning Go and now I think I have a much better understanding of the language, esp. how the module / package works.

Since the repo has been converted to public, I lost write access and here's a PR for #55 based on my fork.

tddschn and others added 30 commits July 31, 2023 16:29
Signed-off-by: Teddy Xinyuan Chen <45612704+tddschn@users.noreply.github.com>
Signed-off-by: Ce Gao <cegao@tensorchord.ai>
Signed-off-by: Ce Gao <cegao@tensorchord.ai>
Signed-off-by: Ce Gao <cegao@tensorchord.ai>
Signed-off-by: Keming <kemingyang@tensorchord.ai>
Signed-off-by: Ce Gao <cegao@tensorchord.ai>
Signed-off-by: Keming <kemingyang@tensorchord.ai>
Signed-off-by: Keming <kemingyang@tensorchord.ai>
Signed-off-by: Ce Gao <cegao@tensorchord.ai>
Signed-off-by: Ce Gao <cegao@tensorchord.ai>
Signed-off-by: Ce Gao <cegao@tensorchord.ai>
Signed-off-by: Ce Gao <cegao@tensorchord.ai>
Signed-off-by: Keming <kemingyang@tensorchord.ai>
Signed-off-by: Keming <kemingyang@tensorchord.ai>
Signed-off-by: Keming <kemingyang@tensorchord.ai>
Signed-off-by: Keming <kemingyang@tensorchord.ai>
Signed-off-by: Keming <kemingyang@tensorchord.ai>
Signed-off-by: Ce Gao <cegao@tensorchord.ai>
Signed-off-by: Ce Gao <cegao@tensorchord.ai>
Signed-off-by: Keming <kemingyang@tensorchord.ai>
Signed-off-by: Keming <kemingyang@tensorchord.ai>
Signed-off-by: Keming <kemingyang@tensorchord.ai>
Signed-off-by: Keming <kemingyang@tensorchord.ai>
Signed-off-by: Keming <kemingyang@tensorchord.ai>
Signed-off-by: Ce Gao <cegao@tensorchord.ai>
Signed-off-by: Ce Gao <cegao@tensorchord.ai>
Signed-off-by: Ce Gao <cegao@tensorchord.ai>
Signed-off-by: Ce Gao <cegao@tensorchord.ai>
Signed-off-by: Ce Gao <cegao@tensorchord.ai>
Signed-off-by: Keming <kemingyang@tensorchord.ai>
gaocegege and others added 8 commits August 5, 2023 11:12
Signed-off-by: Ce Gao <cegao@tensorchord.ai>
Signed-off-by: Ce Gao <cegao@tensorchord.ai>
Signed-off-by: Keming <kemingyang@tensorchord.ai>
Signed-off-by: Keming <kemingyang@tensorchord.ai>
Signed-off-by: Keming <kemingyang@tensorchord.ai>
Signed-off-by: Teddy Xinyuan Chen <45612704+tddschn@users.noreply.github.com>
@gaocegege

Copy link
Copy Markdown
Member

Please rebase the upstream. Thanks!

@tddschn

tddschn commented Aug 9, 2023

Copy link
Copy Markdown
Contributor Author

Should I base it off the 55-mdz-deploy-watch-by-default or the main branch?

To base it off main I think I'll have to open another PR.

To merge to upstream's 55-mdz-deploy-watch-by-default branch, you could try the Squash and Merge button on the GitHub UI. :)

@gaocegege

Copy link
Copy Markdown
Member

Could you close the previous PR, then file a new one in the upstream?

@tddschn

tddschn commented Aug 9, 2023

Copy link
Copy Markdown
Contributor Author

Of course. Please see #137 .

@tddschn tddschn closed this Aug 9, 2023
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.

4 participants