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

[ISSUE #4891] <Add a locking mechanism to solve the problem of repeatedly inserting swagger apidoc&modify the swagger example.> #4892

Merged
merged 88 commits into from Jul 24, 2023

Conversation

lianjunwei
Copy link
Contributor

@lianjunwei lianjunwei commented Jul 24, 2023

detais see: #4891

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.

lianjunwei and others added 30 commits June 9, 2022 01:26
yes
 以 '#' 开始的行将被忽略,而空的提交说明将终止提交。
# Conflicts:
#	shenyu-admin/src/main/java/org/apache/shenyu/admin/service/manager/impl/SwaggerDocParser.java
ok# 以 '#' 开始的行将被忽略,而空的提交说明将终止提交。
… incorrect when micro service parameter uses "@ApiModelProperty".
…c function of the gateway management system.
@codecov-commenter
Copy link

codecov-commenter commented Jul 24, 2023

Codecov Report

Merging #4892 (4f4c806) into master (43e22ed) will decrease coverage by 0.06%.
The diff coverage is 16.94%.

@@             Coverage Diff              @@
##             master    #4892      +/-   ##
============================================
- Coverage     64.23%   64.17%   -0.06%     
- Complexity     8233     8235       +2     
============================================
  Files          1167     1167              
  Lines         34338    34379      +41     
  Branches       3084     3084              
============================================
+ Hits          22057    22063       +6     
- Misses        10522    10556      +34     
- Partials       1759     1760       +1     
Impacted Files Coverage Δ
...va/org/apache/shenyu/admin/model/entity/TagDO.java 67.05% <0.00%> (-2.46%) ⬇️
...ervice/manager/impl/RegisterApiDocServiceImpl.java 4.76% <0.00%> (+0.11%) ⬆️
...ervice/manager/impl/PullSwaggerDocServiceImpl.java 9.67% <2.63%> (-5.03%) ⬇️
...n/java/org/apache/shenyu/admin/model/vo/TagVO.java 80.95% <33.33%> (-3.67%) ⬇️
.../service/manager/impl/LoadServiceDocEntryImpl.java 69.00% <50.00%> (ø)
...ache/shenyu/admin/service/impl/TagServiceImpl.java 61.38% <60.00%> (-0.98%) ⬇️

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

@lianjunwei lianjunwei closed this Jul 24, 2023
@lianjunwei lianjunwei reopened this Jul 24, 2023
@moremind moremind merged commit 7178b82 into apache:master Jul 24, 2023
37 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants