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 admin menu name bug #4805

Merged
merged 1 commit into from
Jul 5, 2023
Merged

[type: fix] fix admin menu name bug #4805

merged 1 commit into from
Jul 5, 2023

Conversation

eurecalulu
Copy link
Contributor

This pull request mainly makes the following changes:

  • Fix the menu name of the document

Before:
中文

After:

中文 英文

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.

@codecov-commenter
Copy link

codecov-commenter commented Jul 5, 2023

Codecov Report

Merging #4805 (485963f) into master (f36157c) will decrease coverage by 0.08%.
The diff coverage is 10.86%.

@@             Coverage Diff              @@
##             master    #4805      +/-   ##
============================================
- Coverage     65.39%   65.32%   -0.08%     
- Complexity     8184     8185       +1     
============================================
  Files          1138     1138              
  Lines         33428    33469      +41     
  Branches       2975     2975              
============================================
+ Hits          21861    21864       +3     
- Misses         9843     9881      +38     
  Partials       1724     1724              
Impacted Files Coverage Δ
...che/shenyu/sync/data/etcd/EtcdSyncDataService.java 62.99% <6.81%> (-12.95%) ⬇️
...henyu/admin/listener/etcd/EtcdDataChangedInit.java 100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes

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

@eurecalulu eurecalulu changed the title fix admin: fix menu name bug [type: fix] fix admin menu name bug Jul 5, 2023
@li-keguo li-keguo merged commit 1c21ed0 into apache:master Jul 5, 2023
33 checks passed
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.

None yet

3 participants