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

Fix json data in upgrade sql. #4374

Merged
merged 13 commits into from
Feb 13, 2023
Merged

Fix json data in upgrade sql. #4374

merged 13 commits into from
Feb 13, 2023

Conversation

u3breeze
Copy link
Contributor

@u3breeze u3breeze commented Feb 10, 2023

Fix json data in upgrade sql.

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 Feb 11, 2023

Codecov Report

Merging #4374 (13aad90) into master (43230a4) will increase coverage by 0.03%.
The diff coverage is 69.73%.

@@             Coverage Diff              @@
##             master    #4374      +/-   ##
============================================
+ Coverage     68.33%   68.36%   +0.03%     
+ Complexity     7515     7511       -4     
============================================
  Files          1020     1020              
  Lines         28801    28813      +12     
  Branches       2568     2568              
============================================
+ Hits          19681    19699      +18     
+ Misses         7580     7578       -2     
+ Partials       1540     1536       -4     
Impacted Files Coverage Δ
...disruptor/subscriber/ApiDocExecutorSubscriber.java 27.27% <0.00%> (+4.19%) ⬆️
.../apache/shenyu/admin/listener/etcd/EtcdClient.java 59.25% <0.00%> (ø)
...va/org/apache/shenyu/admin/model/bean/DocInfo.java 90.00% <ø> (ø)
...ache/shenyu/admin/model/dto/CreateResourceDTO.java 100.00% <ø> (+26.92%) ⬆️
...che/shenyu/admin/model/entity/DashboardUserDO.java 67.41% <0.00%> (+0.74%) ⬆️
.../shenyu/admin/service/impl/AppAuthServiceImpl.java 77.45% <0.00%> (ø)
.../java/org/apache/shenyu/admin/utils/HttpUtils.java 10.16% <0.00%> (ø)
...a/org/apache/shenyu/common/constant/Constants.java 66.66% <ø> (ø)
...yu/common/dto/convert/selector/ZombieUpstream.java 82.05% <ø> (-0.88%) ⬇️
...a/org/apache/shenyu/admin/model/vo/RoleEditVO.java 20.00% <50.00%> (-0.59%) ⬇️
... and 16 more

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

@yu199195 yu199195 merged commit 8e6728b into apache:master Feb 13, 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.

None yet

5 participants