Skip to content

Commit 7b5a201

Browse files
justinmckorca0220
authored andcommitted
Clarify how Gemini should handle conflicting guidelines (flutter#174294)
I ran into a case where [Gemini gave me bad code style advice](flutter#174291 (review)) that it got from Effective Dart even though it conflicted with the Flutter styleguide. This is my attempt to remedy that. Fixes flutter#174293
1 parent 7954351 commit 7b5a201

File tree

1 file changed

+6
-0
lines changed

1 file changed

+6
-0
lines changed

.gemini/styleguide.md

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,12 @@ flutter/flutter repository. It is based on the more comprehensive official
1313
- PR descriptions should include the Pre-launch Checklist from
1414
[the PR template](https://github.com/flutter/flutter/blob/main/.github/PULL_REQUEST_TEMPLATE.md),
1515
with all of the steps completed.
16+
- The most relevant guidelines should take precedence over less relevant
17+
guidelines. For Flutter code, the
18+
[Flutter styleguide](https://github.com/flutter/flutter/blob/main/docs/contributing/Style-guide-for-Flutter-repo.md)
19+
should be followed as the first priority, and
20+
[Effective Dart: Style](https://dart.dev/effective-dart/style)
21+
should only be followed when it does not conflict with the former.
1622

1723
## Review Agent Guidelines
1824

0 commit comments

Comments
 (0)