Skip to content

Conversation

@SbloodyS
Copy link
Member

@SbloodyS SbloodyS commented Jul 9, 2024

Purpose of the pull request

Brief change log

Improve ci check

Verify this pull request

This pull request is code cleanup without any test coverage.

(or)

This pull request is already covered by existing tests, such as (please describe tests).

(or)

This change added tests and can be verified as follows:

(or)

Pull Request Notice

Pull Request Notice

If your pull request contain incompatible change, you should also add it to docs/docs/en/guide/upgrede/incompatible.md

@SbloodyS SbloodyS added this to the 3.3.0 milestone Jul 9, 2024
@SbloodyS SbloodyS added improvement make more easy to user or prompt friendly chore labels Jul 9, 2024
@SbloodyS SbloodyS marked this pull request as draft July 9, 2024 09:54
@SbloodyS SbloodyS marked this pull request as ready for review July 9, 2024 13:47
@github-actions github-actions bot added the e2e e2e test label Jul 9, 2024
@SbloodyS SbloodyS marked this pull request as draft July 9, 2024 14:52
@SbloodyS SbloodyS marked this pull request as ready for review July 10, 2024 00:40
Copy link
Member

@ruanwenjun ruanwenjun left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@caishunfeng caishunfeng left a comment

Choose a reason for hiding this comment

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

LGTM if CI pass

@sonarqubecloud
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
0.0% Coverage on New Code (required ≥ 60%)

See analysis details on SonarCloud

@SbloodyS SbloodyS merged commit bc256a1 into apache:dev Jul 11, 2024
@SbloodyS SbloodyS deleted the improve_ci_check branch July 12, 2024 08:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backend chore CI&CD e2e e2e test improvement make more easy to user or prompt friendly

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants