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 #3180] Remove default datetime of the h2、mysql、pg. #3184

Merged
merged 2 commits into from
Apr 2, 2022

Conversation

ShawnJim
Copy link
Contributor

@ShawnJim ShawnJim commented Apr 2, 2022

ref #3180 .

ShawnJim added 2 commits April 2, 2022 18:57
# Conflicts:
#	shenyu-admin/src/main/resources/sql-script/pg/schema.sql
@codecov-commenter
Copy link

codecov-commenter commented Apr 2, 2022

Codecov Report

Merging #3184 (f1744db) into master (9e163a6) will decrease coverage by 0.04%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #3184      +/-   ##
============================================
- Coverage     57.52%   57.47%   -0.05%     
+ Complexity     4532     4530       -2     
============================================
  Files           730      730              
  Lines         20818    20818              
  Branches       1915     1915              
============================================
- Hits          11976    11966      -10     
- Misses         7792     7803      +11     
+ Partials       1050     1049       -1     
Impacted Files Coverage Δ
...henyu/admin/service/impl/UpstreamCheckService.java 62.85% <0.00%> (-7.15%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9e163a6...f1744db. Read the comment docs.

@yu199195 yu199195 added this to the 2.4.3 milestone Apr 2, 2022
@yu199195 yu199195 merged commit 35e9f98 into apache:master Apr 2, 2022
@ShawnJim ShawnJim deleted the #3180 branch May 21, 2022 12:16
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.

3 participants