I wanted to create a pull request to upgrade go-github to version 29.0.3, however, I've stumbled across issues that I'd like to discuss prior moving forward with implementation.

The issue is that GitHub has deprecated some of the Team API endpoints (marked as Legacy here). After this change go-github has made a braking change in version 29.0.3, where they removed some methods by replacing them with others methods.

This terraform provider currently uses the deleted methods in resource_github_team.go (GetTeam, EditTeam, DeleteTeam) and resource_github_team_repository.go (AddTeamRepo, IsTeamRepo, RemoveTeamRepo).

This makes the upgrade hard because the provider code needs to be updated to use new methods of go-github, but the new methods require extra arguments. For example AddTeamRepo is now replaced by AddTeamRepoByID and AddTeamRepoBySlug. AddTeamRepoByID requires teamId (int) which we do have and orgID (int) which we don't have in state. The other method AddTeamRepoByID requires orgName (string) which we do have in provider config and slug (string) which we do have in state, but not as part of ID.

The situation is even trickier with resource_github_team_repository.go, for example AddTeamRepo func is now replaced by either AddTeamRepoByID or AddTeamRepoBySlug. The AddTeamRepoByID requires orgID (int) which we don't have in state and teamID (int) which is passed as argument into resource. The AddTeamRepoBySlug requires orgName (string), which we do have in provider config, and taem slug (string) which we don't have available.

I'm looking for a guidence to how to solve this issue. Should we increment the schema version and use StateUpgraders to get the missing data into state? Or should we rethink the interface for affected resources?