Skip to content

Conversation

@christopherfujino
Copy link
Contributor

@christopherfujino christopherfujino commented Jul 7, 2023

Add flutter-pub-roller-bot to list of auto_approval_accounts as part of flutter/flutter#129847
@ricardoamador
Copy link
Contributor

By adding this to the $org/.github repo this will apply to all flutter repos. Is that what is intended here?

Copy link
Contributor

@ricardoamador ricardoamador left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@christopherfujino
Copy link
Contributor Author

christopherfujino commented Jul 7, 2023

@ricardoamador I would be ok with moving this to the framework repo config--however, from reading the source, it looked to me like only the org-level config was checked when determining the auto approval accounts via:

  1. https://github.com/flutter/cocoon/blob/main/auto_submit/lib/service/approver_service.dart#L29
  2. https://github.com/flutter/cocoon/blob/main/auto_submit/lib/service/config.dart#L121
  3. https://github.com/flutter/cocoon/blob/main/auto_submit/lib/configuration/repository_configuration_manager.dart#L47
  4. https://github.com/flutter/cocoon/blob/main/auto_submit/lib/configuration/repository_configuration_manager.dart#L68

Am I missing the point where we check the repo-scoped config?

@ricardoamador
Copy link
Contributor

If you add the allow-override flag it will check the local repo. But it might not matter if this bot only operates in the flutter repo autosubmit won't care unless it is in flutter. No big deal.

@christopherfujino
Copy link
Contributor Author

@christopherfujino christopherfujino merged commit b5ac975 into main Jul 7, 2023
@christopherfujino christopherfujino deleted the christopherfujino-patch-1 branch July 7, 2023 22:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants