Skip to content

[Improve] Fix typo on ProcessServiceImpl #15817

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.

Sign up for GitHub

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

Jump to bottom
Merged
qingwli merged 1 commit into from
Apr 9, 2024
Merged

[Improve] Fix typo on ProcessServiceImpl #15817

qingwli merged 1 commit into from
Apr 9, 2024

Conversation

@qingwli
Copy link
Member

@qingwli qingwli commented Apr 9, 2024

Fix typo

@qingwli qingwli requested a review from caishunfeng as a code owner April 9, 2024 05:41
@qingwli qingwli self-assigned this Apr 9, 2024
@qingwli qingwli added the improvement make more easy to user or prompt friendly label Apr 9, 2024
@qingwli qingwli added this to the 3.2.2 milestone Apr 9, 2024
@codecov-commenter
Copy link

codecov-commenter commented Apr 9, 2024 •
edited
Loading

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 39.24%. Comparing base (66df5d4) to head (ba0fc68).

❗ Current head ba0fc68 differs from pull request most recent head 2f33e0f. Consider uploading reports for the commit 2f33e0f to get more accurate results

Additional details and impacted files 
@@             Coverage Diff              @@
##                dev   #15817      +/-   ##
============================================
+ Coverage     39.16%   39.24%   +0.07%     
- Complexity     4888     4908      +20     
============================================
  Files          1323     1323              
  Lines         45110    45184      +74     
  Branches       4796     4810      +14     
============================================
+ Hits          17668    17732      +64     
- Misses        25563    25572       +9     
- Partials       1879     1880       +1

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

ruanwenjun
ruanwenjun approved these changes Apr 9, 2024
View reviewed changes
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

@sonarqubecloud
Copy link

sonarqubecloud bot commented Apr 9, 2024

Please retry analysis of this Pull-Request directly on SonarCloud

@qingwli qingwli merged commit 5050a8e into apache:dev Apr 9, 2024
@qingwli qingwli deleted the fix-typo branch April 9, 2024 06:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ruanwenjun ruanwenjun ruanwenjun approved these changes

@caishunfeng caishunfeng Awaiting requested review from caishunfeng caishunfeng is a code owner

Assignees

@qingwli qingwli

Labels

backend chore improvement make more easy to user or prompt friendly ready-to-merge

Projects

None yet

Milestone

3.2.2

Development

Successfully merging this pull request may close these issues.

3 participants

@qingwli @codecov-commenter @ruanwenjun