Adam Snyder

Yep my opinion on that changed given how long sdk v2 has been supported in terraform core 👍

`group` vs `group_id` is another inconsistency 😭

What would you expect the DELETE operation to do? Should it create a [revert](https://docs.gitlab.com/ee/api/commits.html#revert-a-commit)?

@barundel as an fyi there is an issue and an open PR for templates #333 Let's keep this issue open for discussion a potential project commit resource.

Is this just for the pages feature? We'll keep running the GitLab API in plaintext mode yes?

There is no way to get the shared runner registration token using the API currently. Issue: https://gitlab.com/gitlab-org/gitlab/-/issues/196524

+1 to everything you added. I think you're right about data sources. I can't remember the specific one, but we had a data source change in the last couple months...

That's a neat trick! I didn't know about `TestMain`. (I was going to suggest [build tags](https://mickey.dev/posts/go-build-tags-testing/).) `TestMain` sounds good. 👍

As discussed in community hours today, let's use this issue to track the work on three items: 1. (P1) Use TestMain to gate acceptance tests instead of calling testAccCheck(t) in...

We might need to rethink TestMain. Preventing tests from running at the package level would make our `unit-test` CI job not useful, since we have unit tests and acceptance tests...