Andreas K

@radeksimko I think being the org owner, and not an org member is why you didn't see the bug. > Just for completeness I am an `Owner` of the org...

I've resolved the conflicts with @amitizle PR at https://github.com/carinadigital/terraform-provider-github/tree/add_maintainer I'm currently ensuring all the functionality and tests work.

The current functionality won't work as intended. There is no readMaintainers() like function on teams (https://godoc.org/github.com/google/go-github/github#Team). The current changes to the schema would introduce two places were we would be...

I've sought some guidance at https://github.com/terraform-providers/terraform-provider-github/issues/130 before starting, as the solution might be a little hacky.

I've encountered the same problem. I would suggest that if a duplicate cert is found then it's simply ignored and an error printed to the logs.

Hi @radeksimko I have some proposals for fixing this. It feels wrong to add this discussion to #104 (where I'm actively participating) as there could be multiple solution PRs and...

> I support this behind a feature flag with the old defaults to maintain backwards compatibility. create_default_maintainer defaulting to false is the same as the previous behaviour.