Skip to content

Add Danger check for empty introduced_by_url fields

Niko Belokolodov requested to merge auto_introduced_by_url into master

What does this MR do and why?

I often forget to fill introduced_by_url field while adding a new FF or event/metrics definition.

So I created this Danger check to figure out such fields and suggest url based on MR.

Screenshots or screen recordings

I did commit some config files to test how the plugin works:

Screen_Shot_2022-08-17_at_3.58.14_PM

Screen_Shot_2022-08-17_at_3.58.05_PM

Screen_Shot_2022-08-17_at_3.57.53_PM

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Niko Belokolodov

Merge request reports