Skip to content

Implement readFrom without the logger argument#19

Merged
Borewit merged 1 commit into
mainfrom
do-not-require-logger
Mar 4, 2023
Merged

Implement readFrom without the logger argument#19
Borewit merged 1 commit into
mainfrom
do-not-require-logger

Conversation

@Borewit

@Borewit Borewit commented Mar 4, 2023

Copy link
Copy Markdown
Owner

Uses the SpecificPlaylistProvider class logger instead.

@Borewit Borewit self-assigned this Mar 4, 2023
Comment thread src/main/java/christophedelory/playlist/hypetape/HypetapeProvider.java Outdated
@sonatype-lift

sonatype-lift Bot commented Mar 4, 2023

Copy link
Copy Markdown

🛠 Lift Auto-fix

Some of the Lift findings in this PR can be automatically fixed. You can download and apply these changes in your local project directory of your branch to review the suggestions before committing.1

# Download the patch
curl https://lift.sonatype.com/api/patch/github.com/Borewit/lizzy/19.diff -o lift-autofixes.diff

# Apply the patch with git
git apply lift-autofixes.diff

# Review the changes
git diff

Want it all in a single command? Open a terminal in your project's directory and copy and paste the following command:

curl https://lift.sonatype.com/api/patch/github.com/Borewit/lizzy/19.diff | git apply

Once you're satisfied, commit and push your changes in your project.

Footnotes

  1. You can preview the patch by opening the patch URL in the browser.

Uses the `SpecificPlaylistProvider` class logger instead.
@Borewit
Borewit force-pushed the do-not-require-logger branch from 378909e to 4083788 Compare March 4, 2023 14:11
@Borewit Borewit added the improvement Improvement to existing functionality label Mar 4, 2023
@Borewit
Borewit merged commit 7250374 into main Mar 4, 2023
@Borewit
Borewit deleted the do-not-require-logger branch March 5, 2023 13:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

improvement Improvement to existing functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant