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
Fail when creating a Scope with a null-parent #1354. #1377
Conversation
- Prevent subsequent lookup of values in the parent-scope, from failing. - In case of failure: Preserve stack trace to uncover root cause.
ed15ba7
to
4f00bc7
Compare
Not sure what codecov is trying to tell me. Please let me know if there is something I should fix. |
Hi @Treehopper Thanks for creating this PR. A member of the Liquibase team will take a look at your contribution and may suggest We’ll let you know when it’s ready to move to the next step or if any changes are needed. |
I pushed a merge conflict fix to your branch, along with moving your test to ScopeTest. Marking this as NoManualTesting since it's a null-check that should only be hittable by libraries using Liquibase, not by our code. |
@@ -26,6 +26,8 @@ | |||
import java.nio.charset.Charset; | |||
import java.util.*; | |||
|
|||
import static java.util.Objects.requireNonNull; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like this is now unused.
See #1354
name: Pull Request
about: Create a report to help us improve
title: ''
labels: Status:Discovery
assignees: ''
Environment
Liquibase Version:
Liquibase Integration & Version: <Pick one: CLI, maven, gradle, spring boot, servlet, etc.>
Liquibase Extension(s) & Version:
Database Vendor & Version:
Operating System Type & Version:
Pull Request Type
Description
A clear and concise description of the issue being addressed. Additional guidance here.
Steps To Reproduce
List the steps to reproduce the behavior.
Actual Behavior
A clear and concise description of what happens in the software before this pull request.
Expected/Desired Behavior
A clear and concise description of what happens in the software after this pull request.
Screenshots (if appropriate)
If applicable, add screenshots to help explain your problem.
Additional Context
Add any other context about the problem here.
Fast Track PR Acceptance Checklist:
<
!--- If you're unsure about any of these, just ask us in a comment. We're here to help|width=200,height=183!
-->
Need Help?
Come chat with us on our discord channel
Dev Handoff Notes (Internal Use)
Links
Testing
Dev Verification
Code review and automated tests
┆Issue is synchronized with this Jira Bug by Unito