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

EntityResolver should check the classpath #1621

Merged
merged 2 commits into from
Jan 7, 2021
Merged

EntityResolver should check the classpath #1621

merged 2 commits into from
Jan 7, 2021

Conversation

nvoxland
Copy link
Contributor

@nvoxland nvoxland commented Jan 6, 2021

Even if the standard ResourceAccessor does not include it.

Fixes #1609

@codecov
Copy link

codecov bot commented Jan 6, 2021

Codecov Report

Merging #1621 (800d1ea) into 4.3.x (18202bc) will increase coverage by 0.02%.
The diff coverage is 34.49%.

Impacted file tree graph

@@             Coverage Diff              @@
##              4.3.x    #1621      +/-   ##
============================================
+ Coverage     49.59%   49.62%   +0.02%     
- Complexity     7862     7880      +18     
============================================
  Files           757      759       +2     
  Lines         37024    37133     +109     
  Branches       6699     6726      +27     
============================================
+ Hits          18363    18427      +64     
- Misses        16231    16248      +17     
- Partials       2430     2458      +28     
Impacted Files Coverage Δ Complexity Δ
...uibase-core/src/main/java/liquibase/Liquibase.java 40.07% <0.00%> (+0.19%) 58.00 <0.00> (+1.00)
.../main/java/liquibase/change/AbstractSQLChange.java 82.03% <0.00%> (ø) 29.00 <0.00> (ø)
...quibase/change/core/ExecuteShellCommandChange.java 15.38% <ø> (ø) 13.00 <0.00> (ø)
.../java/liquibase/database/core/Ingres9Database.java 26.31% <0.00%> (ø) 8.00 <0.00> (ø)
...main/java/liquibase/datatype/core/BooleanType.java 19.44% <0.00%> (ø) 10.00 <0.00> (ø)
...e/src/main/java/liquibase/example/StartH2Main.java 0.00% <0.00%> (ø) 0.00 <0.00> (ø)
.../liquibase/hub/listener/HubChangeExecListener.java 14.81% <0.00%> (-0.24%) 11.00 <0.00> (ø)
...n/java/liquibase/integration/commandline/Main.java 25.02% <0.00%> (-0.03%) 138.00 <0.00> (ø)
.../liquibase/integration/spring/SpringLiquibase.java 0.00% <0.00%> (ø) 0.00 <0.00> (ø)
.../java/liquibase/parser/ChangeLogParserFactory.java 93.33% <ø> (ø) 13.00 <0.00> (ø)
... and 49 more

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 d3f1efa...800d1ea. Read the comment docs.

@molivasdat molivasdat linked an issue Jan 6, 2021 that may be closed by this pull request
@molivasdat molivasdat merged commit 57f8647 into 4.3.x Jan 7, 2021
@suryaaki2 suryaaki2 deleted the LB-1079 branch July 30, 2021 15:28
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.

CORE-3198 Regression - prefer internal XSD usage
2 participants