Skip to content

test:alter branch build#29631

Merged
feici02 merged 5 commits into
mainfrom
test/main_ci_sh
Jan 22, 2025
Merged

test:alter branch build#29631
feici02 merged 5 commits into
mainfrom
test/main_ci_sh

Conversation

@happyguoxy
Copy link
Copy Markdown
Contributor

@happyguoxy happyguoxy commented Jan 22, 2025

Description

Please briefly describe the code changes in this pull request.

Checklist

Please check the items in the checklist if applicable.

  • Is the user manual updated?
  • Are the test cases passed and automated?
  • Is there no significant decrease in test coverage?

Copy link
Copy Markdown
Member

@feici02 feici02 left a comment

Choose a reason for hiding this comment

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

Some of the logics in this script are among the the lines of several functions, can they be moved to together?

@happyguoxy
Copy link
Copy Markdown
Contributor Author

Some of the logics in this script are among the the lines of several functions, can they be moved to together?

ok,all case have been modified.

Copy link
Copy Markdown
Member

@feici02 feici02 left a comment

Choose a reason for hiding this comment

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

LGTM

@feici02 feici02 merged commit 95985fd into main Jan 22, 2025
@minhuinie minhuinie deleted the test/main_ci_sh branch February 4, 2026 03:01
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