Skip to content

Respect disabled global filter data setting - #544

Merged
ankane merged 1 commit into
ankane:masterfrom
OskarEichler:songstats/filter-data-false
Sep 6, 2026
Merged

ankane merged 1 commit into
ankane:masterfrom
OskarEichler:songstats/filter-data-false

Conversation

@OskarEichler

Copy link
Copy Markdown
Contributor

Summary

  • return the configured global filter_data value instead of testing only whether the key exists
  • preserve per-database precedence and the legacy default

A global filter_data: false currently becomes true because Hash#key? is returned from the value branch.

Verification

  • focused configuration model covers per-database, global false/true, and default cases
  • full local suite: 119 runs, 2,235 assertions, 0 failures/errors
  • Ruby syntax checks: 32/32 files
  • packaged gem contents: 75 files

@ankane
ankane merged commit 8c11994 into ankane:master Sep 6, 2026
@ankane

ankane commented Sep 6, 2026

Copy link
Copy Markdown
Owner

Thanks @OskarEichler

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