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

Format spring package. #5597

Merged
merged 5 commits into from Mar 5, 2024
Merged

Format spring package. #5597

merged 5 commits into from Mar 5, 2024

Conversation

asashour
Copy link
Contributor

The current code sometimes uses 2-space indentation and 4-space one.

I think it makes sense to format the spring package (for now) for 4-space indentation, as a preparation to work on a PR for that package.

Please let me know if 2-space one is preferred.

@MalloD12 MalloD12 self-requested a review February 29, 2024 18:13
@MalloD12 MalloD12 self-assigned this Feb 29, 2024
@MalloD12 MalloD12 changed the base branch from master to add-db2-history-test February 29, 2024 18:38
@MalloD12 MalloD12 changed the base branch from add-db2-history-test to master February 29, 2024 18:38
Copy link
Contributor

@MalloD12 MalloD12 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved.

Formatting changes look good to me. I removed a deprecated set method and its related get method as well, since they were not used and there was a codeQL scan issue reported (due to an unused argument).

No builds or test checks issues reported.

@@ -60,72 +62,88 @@
protected final Logger log = Scope.getCurrentScope().getLog(SpringLiquibase.class);
protected String beanName;

protected ResourceLoader resourceLoader;
@Getter
@Setter

Check notice

Code scanning / CodeQL

Missing Override annotation Note

This method overrides
ResourceLoaderAware.setResourceLoader
; it is advisable to add an Override annotation.
@filipelautert filipelautert added this to the 1NEXT milestone Mar 5, 2024
@filipelautert filipelautert merged commit 7975e86 into liquibase:master Mar 5, 2024
30 of 33 checks passed
@asashour asashour deleted the format branch March 5, 2024 12:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

3 participants