Skip to content

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
Collaborator

@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

This method overrides [ResourceLoaderAware.setResourceLoader](1); 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
@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
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants