Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

#106 and #107 added HTML and CSS formatter #232

Merged
merged 3 commits into from
Oct 18, 2022
Merged

#106 and #107 added HTML and CSS formatter #232

merged 3 commits into from
Oct 18, 2022

Conversation

funfried
Copy link
Owner

No description provided.

@sonatype-lift
Copy link

sonatype-lift bot commented Oct 18, 2022

⚠️ 3 God Classes were detected by Lift in this project. Visit the Lift web console for more details.


Preferences preferences = Settings.getActivePreferences(document);
if (isUseFormatterIndentationSettings(preferences)) {
if (!isExpandTabToSpaces(document, preferences) && preferences.getBoolean(Settings.OVERRIDE_TAB_SIZE, true)) {
Copy link

Choose a reason for hiding this comment

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

NULL_DEREFERENCE: object returned by isExpandTabToSpaces(document,preferences) could be null and is dereferenced at line 149.


ℹ️ Learn about @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.


Was this a good recommendation?
[ 🙁 Not relevant ] - [ 😕 Won't fix ] - [ 😑 Not critical, will fix ] - [ 🙂 Critical, will fix ] - [ 😊 Critical, fixing now ]

@funfried funfried merged commit da5c4e6 into master Oct 18, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant