Skip to content

fix: FEAT: Notification sound toggle (#1547)#1604

Open
BWM0223 wants to merge 1 commit into
floatpane:masterfrom
BWM0223:fix-1547-1782077140
Open

fix: FEAT: Notification sound toggle (#1547)#1604
BWM0223 wants to merge 1 commit into
floatpane:masterfrom
BWM0223:fix-1547-1782077140

Conversation

@BWM0223

@BWM0223 BWM0223 commented Jun 21, 2026

Copy link
Copy Markdown

Fixes #1547

Automated by REAPR

@BWM0223 BWM0223 requested a review from a team as a code owner June 21, 2026 21:25

@floatpanebot floatpanebot left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Hi @BWM0223! Please fix the following issues with your PR:

  • Title: Is too long (44 characters). The PR title must be strictly under 40 characters.
  • Body: Missing the ## What? or ## Why? headings required by the PR template.

@floatpanebot floatpanebot added bug Something isn't working size/S Diff: 11–50 lines labels Jun 21, 2026
@floatpanebot

Copy link
Copy Markdown
Member

Benchmark report — no significant change

Metrics worse: 0 · better: 0 (threshold: ±3%).

benchstat output
goos: linux
goarch: amd64
pkg: github.com/floatpane/matcha/backend
cpu: AMD EPYC 7763 64-Core Processor                
                           │    old.txt    │               new.txt                │
                           │    sec/op     │    sec/op      vs base               │
ParseSearchQuery_Simple-4    3.435µ ± 189%   3.188µ ± 493%        ~ (p=0.937 n=6)
ParseSearchQuery_Complex-4   8.436µ ±  34%   9.201µ ±  53%        ~ (p=0.485 n=6)
TokenizeSearchQuery-4        5.415µ ± 125%   8.828µ ±  67%        ~ (p=0.180 n=6)
geomean                      5.394µ          6.373µ         +18.16%

                           │  old.txt   │              new.txt               │
                           │    B/op    │    B/op     vs base                │
ParseSearchQuery_Simple-4    26.00 ± 0%   26.00 ± 0%       ~ (p=1.000 n=6) ¹
ParseSearchQuery_Complex-4   762.0 ± 0%   762.0 ± 0%       ~ (p=1.000 n=6) ¹
TokenizeSearchQuery-4        176.0 ± 0%   176.0 ± 0%       ~ (p=1.000 n=6) ¹
geomean                      151.6        151.6       +0.00%
¹ all samples are equal

                           │  old.txt   │              new.txt               │
                           │ allocs/op  │ allocs/op   vs base                │
ParseSearchQuery_Simple-4    2.000 ± 0%   2.000 ± 0%       ~ (p=1.000 n=6) ¹
ParseSearchQuery_Complex-4   23.00 ± 0%   23.00 ± 0%       ~ (p=1.000 n=6) ¹
TokenizeSearchQuery-4        9.000 ± 0%   9.000 ± 0%       ~ (p=1.000 n=6) ¹
geomean                      7.453        7.453       +0.00%
¹ all samples are equal

pkg: github.com/floatpane/matcha/tui
                    │   old.txt    │              new.txt               │
                    │    sec/op    │    sec/op     vs base              │
LogPanelView-4        172.5µ ±  8%   182.2µ ±  7%       ~ (p=0.240 n=6)
SearchOverlayView-4   190.0µ ± 14%   185.6µ ± 13%       ~ (p=0.818 n=6)
InboxConstruction-4   986.0µ ± 41%   971.3µ ± 19%       ~ (p=0.818 n=6)
geomean               318.5µ         320.2µ        +0.55%

                    │    old.txt    │               new.txt                │
                    │     B/op      │     B/op       vs base               │
LogPanelView-4        44.67Ki ± 51%   44.67Ki ± 51%        ~ (p=1.000 n=6)
SearchOverlayView-4   33.18Ki ± 69%   56.14Ki ± 41%        ~ (p=0.864 n=6)
InboxConstruction-4   874.2Ki ±  0%   874.3Ki ±  0%        ~ (p=0.667 n=6)
geomean               109.0Ki         129.9Ki        +19.16%

                    │   old.txt   │              new.txt              │
                    │  allocs/op  │  allocs/op   vs base              │
LogPanelView-4         714.0 ± 0%    714.0 ± 0%       ~ (p=1.000 n=6)
SearchOverlayView-4    924.0 ± 0%    926.0 ± 0%       ~ (p=0.470 n=6)
InboxConstruction-4   3.478k ± 0%   3.478k ± 0%       ~ (p=0.457 n=6)
geomean               1.319k        1.320k       +0.08%

auto-generated by benchmarks.yml

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working size/S Diff: 11–50 lines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

FEAT: Notification sound toggle

2 participants