Skip to content

Log smart playlist create msg as info log - #4861

Merged
sampsyo merged 2 commits into
beetbox:masterfrom
JOJ0:splupdate_create_log
Jul 25, 2023
Merged

Log smart playlist create msg as info log#4861
sampsyo merged 2 commits into
beetbox:masterfrom
JOJ0:splupdate_create_log

Conversation

@JOJ0

@JOJ0 JOJ0 commented Jul 25, 2023

Copy link
Copy Markdown
Member

Log the creation of a smart playlist as info instead of debug log level.

Description

In an explicit run (not while importing) it doesn't hurt to get some progress to the UI since creating/updating smart playlists can take a singinficant amount of time.

To Do

  • Documentation.
  • Changelog.
  • Tests.

JOJ0 added 2 commits July 25, 2023 07:52
instead of debug log. In an explicit run (not while importing) it doesn't hurt
to get some progress to the UI.
@JOJ0 JOJ0 changed the title Log smartplaylist create msg as info log Log smart playlist create msg as info log Jul 25, 2023
@sampsyo

sampsyo commented Jul 25, 2023

Copy link
Copy Markdown
Member

Good call! And you're right; this won't disrupt automatic (beet import) runs.

@sampsyo
sampsyo merged commit 0cc52fe into beetbox:master Jul 25, 2023
Louson pushed a commit to Louson/beets that referenced this pull request Sep 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants