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

[type:fix] fix lack separator of path of ApiDoc #4528

Merged
merged 2 commits into from
Apr 2, 2023

Conversation

tian-pengfei
Copy link
Contributor

Make sure that:

  • You have read the contribution guidelines.
  • You submit test cases (unit or integration tests) that back your changes.
  • Your local test passed ./mvnw clean install -Dmaven.javadoc.skip=true.

fix #4527

before:

image

after:

image

@codecov-commenter
Copy link

codecov-commenter commented Apr 2, 2023

Codecov Report

Merging #4528 (9bdc045) into master (4491cb8) will decrease coverage by 0.04%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #4528      +/-   ##
============================================
- Coverage     68.46%   68.43%   -0.04%     
+ Complexity     7631     7623       -8     
============================================
  Files          1028     1028              
  Lines         29440    29440              
  Branches       2666     2666              
============================================
- Hits          20157    20147      -10     
- Misses         7690     7698       +8     
- Partials       1593     1595       +2     

see 3 files with indirect coverage changes

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

@loongs-zhang loongs-zhang added this to the 2.6.0 milestone Apr 2, 2023
@tian-pengfei tian-pengfei merged commit 9b726ee into apache:master Apr 2, 2023
@tian-pengfei tian-pengfei deleted the fix_lack_separator branch April 3, 2023 04:45
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.

[bug] path of APIDOC lack '/'
3 participants