Skip to content

[Improvement][doc]fix typo in readme#14878

Merged
Radeity merged 2 commits intoapache:devfrom
zjunothing:minor-fix
Sep 7, 2023
Merged

[Improvement][doc]fix typo in readme#14878
Radeity merged 2 commits intoapache:devfrom
zjunothing:minor-fix

Conversation

@zjunothing
Copy link
Contributor

Purpose of the pull request

Fix a typo, and by the way just try my first ci to get familiar with PR process

This pull request is code cleanup without any test coverage.

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, glad to see you in ds.

@codecov-commenter
Copy link

codecov-commenter commented Sep 7, 2023

Codecov Report

Merging #14878 (06df444) into dev (5062aa1) will decrease coverage by 0.03%.
The diff coverage is n/a.

❗ Current head 06df444 differs from pull request most recent head 1429b1b. Consider uploading reports for the commit 1429b1b to get more accurate results

@@             Coverage Diff              @@
##                dev   #14878      +/-   ##
============================================
- Coverage     38.89%   38.87%   -0.03%     
+ Complexity     4585     4583       -2     
============================================
  Files          1232     1232              
  Lines         43237    43237              
  Branches       4772     4772              
============================================
- Hits          16817    16808       -9     
- Misses        24560    24570      +10     
+ Partials       1860     1859       -1     

see 2 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@sonarqubecloud
Copy link

sonarqubecloud bot commented Sep 7, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@Radeity Radeity merged commit f3a4842 into apache:dev Sep 7, 2023
zhongjiajie pushed a commit that referenced this pull request Oct 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants