I don't think we should change the names of the resources as that would cause recreations of resources which breaks backwards compatibility.

backwards compatibility doesn't seem like it's much of a concern in other modules though? major changes to input vars on other cloudposse repos get the release labelled as a breaking change and put out with a minor version bump. eg: https://github.com/cloudposse/terraform-aws-ecs-alb-service-task/releases/tag/0.57.0

these changes don't even change the input vars. it would trigger a rebuild yes, but it also wouldn't require code changes on the user's part.

since this is a permanent change in the way aws treats github repos going forward, i think that getting prepared for a move away from webhooks being the default would be good to do before it becomes a last minute thing.

This pull request is now in conflict. Could you fix it @inhumantsar? 🙏