Enable EE acceptance tests in pull requests

Adam Snyderrequested to merge
run-ee-tests-in-prs into master

Description

closes #426 (closed) closes #801 (closed)

🥳 🦖 🎆

This will take effect after we merge it, without PR authors even needing to rebase.

I tested out these workflow changes in my fork and opened a PR from yet another fork: https://github.com/armsnyder/terraform-provider-gitlab/pull/35

Additionally I checked what happens on Dependabot PRs, and they correctly skip the EE test on the push workflow and run it on the PR workflow: https://github.com/armsnyder/terraform-provider-gitlab/pull/32 (Failure is unrelated to this change.)

And of course the master build still works.

PR Checklist

  • Resource attributes match 1:1 the names and structure of the API resource in the GitLab API documentation.
  • Examples are updated with:
    • A *.tf file for the resource/s with at least one usage example
    • A *.sh file for the resource/s with an import example (if applicable)
  • New resources have at minimum a basic test with three steps:
    • Create the resource
    • Update the attributes
    • Import the resource
  • No new //lintignore comments that came from copied code. Linter rules are meant to be enforced on new code.

Merge request reports