From de97a05095cca5fd923d2f911c27aa5586a23e2e Mon Sep 17 00:00:00 2001 From: Kate Lovett Date: Wed, 21 Feb 2024 20:28:02 -0600 Subject: [PATCH] Update PR template for dart fix (#143879) I don't know if this is feasible, but figured it was worth seeing. Often times dart fixes are forgotten as part of the code review process. I know some people ignore the PR template, but considered maybe a mention was warranted. I've made the change reflected below: --- .github/PULL_REQUEST_TEMPLATE.md | 2 ++ 1 file changed, 2 insertions(+) diff --git a/.github/PULL_REQUEST_TEMPLATE.md b/.github/PULL_REQUEST_TEMPLATE.md index 43b7da0af1..f33ad35e44 100644 --- a/.github/PULL_REQUEST_TEMPLATE.md +++ b/.github/PULL_REQUEST_TEMPLATE.md @@ -13,6 +13,7 @@ - [ ] I listed at least one issue that this PR fixes in the description above. - [ ] I updated/added relevant documentation (doc comments with `///`). - [ ] I added new tests to check the change I am making, or this PR is [test-exempt]. +- [ ] I followed the [breaking change policy] and added [Data Driven Fixes] where supported. - [ ] All existing and new tests are passing. If you need help, consider asking for advice on the #hackers-new channel on [Discord]. @@ -27,3 +28,4 @@ If you need help, consider asking for advice on the #hackers-new channel on [Dis [flutter/tests]: https://github.com/flutter/tests [breaking change policy]: https://github.com/flutter/flutter/wiki/Tree-hygiene#handling-breaking-changes [Discord]: https://github.com/flutter/flutter/wiki/Chat +[Data Driven Fixes]: https://github.com/flutter/flutter/wiki/Data-driven-Fixes