Skip to content

Adjust CheckFewHitsounds verify check based on feedback#37466

Merged
bdach merged 3 commits into
ppy:masterfrom
Hiviexd:verify/fix-few-hs-check
May 11, 2026
Merged

Adjust CheckFewHitsounds verify check based on feedback#37466
bdach merged 3 commits into
ppy:masterfrom
Hiviexd:verify/fix-few-hs-check

Conversation

@Hiviexd
Copy link
Copy Markdown
Member

@Hiviexd Hiviexd commented Apr 21, 2026

commit 1 makes sure breaks are skipped in calculation to avoid false positives (reported internally)

commit 2 makes the check only available in osu! and osu!catch as it's not relevant in:

  • osu!taiko, as it usually only triggers on valid mono-mapping (section with all dons for example).
  • osu!mania, given hitsounding is optional there.
  • (brief internal discussion)

@bdach bdach self-requested a review May 7, 2026 10:43
Comment thread osu.Game/Rulesets/Edit/Checks/CheckFewHitsounds.cs Outdated
spinners for osu!, banana showers for osu!catch
@pull-request-size pull-request-size Bot added size/L and removed size/M labels May 8, 2026
@bdach bdach merged commit 107e875 into ppy:master May 11, 2026
8 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants