-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Improved readability in the createCDILiquibaseConfig method. #3316
Conversation
@musttafayildirim Can you please provide some of the requested details in the Description? I understand this isn't a major change so some of the questions might not apply, but more context helps us with review, release notes, etc. Thanks much! |
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.
This PR improves the SchemesCDIConfigBuilder method to improve readability. This change introduces an end
variable to indicate the time when a scan completes, allowing developers to add a watch as they step through the code. Prior to this change, the end was buried in a log.info
statement.
- Test harness failure unrelated to this change.
- No additional testing required.
APPROVED
Impact
Description
Adds new line breaks to a code that seems packed up. No functionality change, just code readability.
Things to be aware of
None
Things to worry about
None
Additional Context
None