Repository navigation
Conversation
zanderso
left a comment
There was a problem hiding this comment.
I think the PRs might not be split up quite right? It looks like there's some overlap between this PR and number one in the sequence?
| @@ -1,4 +1,5 @@ | |||
| import 'dart:convert'; | |||
There was a problem hiding this comment.
code needs a copyright header. here and below.
Yes, from the PR description: Based off of #4 - should be reviewed first |
jtmcdole
left a comment
There was a problem hiding this comment.
lgtm, but waiting on Zach's change request
Based off of #4 - should be reviewed first
Part 3 of 4
Full Picture: #3
Towards flutter/flutter#186415
RFC: flutter/rfc#36
Seeds the initial Flutter Contributor Ladder roster and adds validation and generation tooling:
data/contributors.yamlas the single source of truth for non-Maintainer ladder tiers (Committer,Reviewer,Emeritus) and generatesROSTER.mdandEMERITUS.md.tool/validate_roster.dartandtool/generate_roster.dart(with--checkmode) to validate schema integrity, bot/maintainer exclusions, and markdown synchronization.tool/seed_initial_data.dartand unit tests intest/roster_validation_test.dart, and wires roster validation into.github/workflows/validate.yml.Pre-launch Checklist
///).If you need help, consider asking for advice on the #hackers-new channel on Discord.