-
Notifications
You must be signed in to change notification settings - Fork 8
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
[Integration tests]: manually get the current commit for the action w…
…hen bootstrapping the project. (#405) This PR does two things: * Moves testing to a reusable workflow which is now shared between the `main`, `pr`, and `release` workflows. * Updates the integration tests to use the version of the grafana-app-sdk that matches the code the action is being run against (using a `replace` directive in the `go.mod`, as using `GITHUB_SHA` doesn't cooperate with `go get` in PR builds). Currently, the branch protection settings for `main` require that an action named `test` complete successfully to merge (which no longer exists the same way, so this blocks indefinitely), once this PR is approved I will update the branch protection settings to require `test / unit-test`, `test / integration-test`, and `test / build` instead, then merge this PR so other PR's will need to merge in from main to get correct testing.
- Loading branch information
1 parent
783cd4d
commit 73db82e
Showing
5 changed files
with
156 additions
and
395 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.