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

refactor: Upgrade gsctl and modify files associated with gsctl #3149

Merged
merged 2 commits into from
Aug 31, 2023

Conversation

FLYLX
Copy link
Contributor

@FLYLX FLYLX commented Aug 28, 2023

What do these changes do?

Added the test method to gsctl.
Modify files associated with gsctl.

Related issue number

issue #3074

@FLYLX FLYLX force-pushed the 3074-upgrade-gsctl-test branch from 5633b17 to 7743ea5 Compare August 28, 2023 05:54
@yecol yecol requested a review from siyuan0322 August 28, 2023 07:04
@FLYLX FLYLX force-pushed the 3074-upgrade-gsctl-test branch from 7743ea5 to 1f898d5 Compare August 28, 2023 08:26
@@ -76,7 +77,7 @@ function test_analytical-java {

function test_interactive {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unify the indentation of functions in all scripts

@FLYLX FLYLX force-pushed the 3074-upgrade-gsctl-test branch 3 times, most recently from 57de977 to 6d76af0 Compare August 31, 2023 11:23
@FLYLX FLYLX force-pushed the 3074-upgrade-gsctl-test branch from 6d76af0 to 6a2837c Compare August 31, 2023 13:27
@lidongze0629 lidongze0629 merged commit 5d6cb49 into alibaba:main Aug 31, 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.

3 participants