Adding IsaacG to the Go maintainers

Exercism
1

@andrerfcsantos would you support having me join to Go maintainers group? I noticed there’s some Go work that could use some extra hands.

3 Likes
2

Yeah, for sure! Maybe something that @iHiD can do?

1 Like
3

I think Erik also has admin access to the GH team membership.

4

Thanks!

+cc @ErikSchierboom for maintainer update.

5

You’ve been added to the team @IsaacG.

6

Thanks, Erik!

7

The Go repo is marked as maintained-autonomous and I don’t seem to have merge permissions. Does something need to be updated for me to be able to merge PRs or will that eventually update automatically?

8

I’ll need to run a script for that. I’ll do that later

9

Ah, the logic dictates it should still be maintained-autonomous:

elsif team_members.all? {|m| cross_track_maintainers_team_members.include?(m) || admin_team_members.include?(m)}
    status = :maintained_autonomous
1 Like
10

You are member of the team so you should be able to merge PRs.

1 Like
11

https://github.com/exercism/go/pull/2898

image
image985×780 61.8 KB

Shouldn’t I see a merge button here? Am I missing something?

12

You should! Very odd

13

Ultra-trivial suggestion, but I’ve seen similar GH problems where a page refresh sorts it out. The mysterious effects of hitting F5…

14

Force refresh didn’t help so I tried to bypass the UI.

» gh pr merge --squash https://github.com/exercism/go/pull/2898
GraphQL: IsaacG does not have the correct permissions to execute `MergePullRequest` (mergePullRequest)
15

Try again!

16 
» gh pr merge --squash https://github.com/exercism/go/pull/2898
✓ Squashed and merged pull request exercism/go#2898 (build(deps): bump actions/setup-go from 6.1.0 to 6.2.0)

Thank you! What was missing?

1 Like
17

Apparently the Go track is configured differently and I just gave you personal permissions on the repo just like Andre has

2 Likes
18

@andrerfcsantos I queued up for PRs for your review ;)
Or I can merge them unreviewed, if you don’t have time to review them.