Skip to content
Snippets Groups Projects

Improve matching patterns in "settings_sections" Dangerfile

Merged Thomas Hutterer requested to merge 463524-also-check-section-component into master

What does this MR do and why?

This is a small follow-up to !161020 (merged) to close #463524 (closed).

  • Now matches both SettingsBlock and SettingsSection
  • Now ignores templates in admin and profiles folders. These settings aren't searchable.

¹: I forgot that we actually have two different components for this. One is "Title + Expand/Collapse", the other is "Sticky title"... Sorry for not catching this before opening the first MR :see_no_evil:

How to set up and validate locally

  • Checkout this branch, delete a searchable settings section and run lefthook run pre-push to see the new Danger output.
  • Or check !162035 (closed) as a demo MR with such a warning in the Danger comment.
Edited by Thomas Hutterer

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading