Skip to content
This repository has been archived by the owner on Jun 30, 2023. It is now read-only.

V3.0.0 #34

Merged
merged 5 commits into from
Jun 25, 2023
Merged

V3.0.0 #34

merged 5 commits into from
Jun 25, 2023

Conversation

pangzixiang
Copy link
Contributor

No description provided.

@@ -24,6 +29,20 @@
@Slf4j
Copy link

Choose a reason for hiding this comment

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

5% of developers fix this issue

UnnecessarilyFullyQualified: This fully qualified name is unambiguous to the compiler if imported.


Suggested change
@Slf4j
SuppressWarnings

❗❗ 2 similar findings have been found in this PR

🔎 Expand here to view all instances of this finding
File Path Line Number
src/main/java/io/github/pangzixiang/whatsit/vertx/core/ApplicationRunner.java 29
src/main/java/io/github/pangzixiang/whatsit/vertx/core/ApplicationRunner.java 29

Visit the Lift Web Console to find more details in your report.


ℹ️ Expand to see all @sonatype-lift commands

You can reply with the following commands. For example, reply with @sonatype-lift ignoreall to leave out all findings.

Command Usage
@sonatype-lift ignore Leave out the above finding from this PR
@sonatype-lift ignoreall Leave out all the existing findings from this PR
@sonatype-lift exclude <file|issue|path|tool> Exclude specified file|issue|path|tool from Lift findings by updating your config.toml file

Note: When talking to LiftBot, you need to refresh the page to see its response.
Click here to add LiftBot to another repo.

@pangzixiang pangzixiang merged commit ad071d1 into master Jun 25, 2023
2 of 3 checks passed
@pangzixiang pangzixiang deleted the v3.0.0 branch June 25, 2023 07:09
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant