Skip to content
This repository was archived by the owner on Oct 14, 2024. It is now read-only.

test: fix make test target #1149

Merged
merged 2 commits into from
Feb 1, 2024
Merged

test: fix make test target #1149

merged 2 commits into from
Feb 1, 2024

Conversation

chrisgacsal
Copy link
Contributor

@chrisgacsal chrisgacsal commented Feb 1, 2024

Description

  • make sure to run go test ... for all the modules in the repository
  • use the -short flag for go test to exclude complex integration/end-to-end tests by default

Type of Change

[x] Bug Fix
[ ] New Feature
[ ] Breaking Change
[ ] Refactor
[ ] Documentation
[x] Other (please describe)

Checklist

  • I have read the contributing guidelines
  • Existing issues have been referenced (where applicable)
  • I have verified this change is not present in other open pull requests
  • Functionality is documented
  • All code style checks pass
  • New code contribution is covered by automated tests
  • All new and existing tests pass

Make sure that we run `go test` for all modules in the repository.
@chrisgacsal chrisgacsal requested a review from a team as a code owner February 1, 2024 12:49
@chrisgacsal chrisgacsal self-assigned this Feb 1, 2024
Copy link

github-actions bot commented Feb 1, 2024

Hey!

Your images are ready:

  • ghcr.io/openclarity/vmclarity-apiserver-dev:pr1149-eb4f1ad3a3dd258131b0a366241bb2b4aa642b56
  • ghcr.io/openclarity/vmclarity-orchestrator-dev:pr1149-eb4f1ad3a3dd258131b0a366241bb2b4aa642b56
  • ghcr.io/openclarity/vmclarity-ui-backend-dev:pr1149-eb4f1ad3a3dd258131b0a366241bb2b4aa642b56
  • ghcr.io/openclarity/vmclarity-ui-dev:pr1149-eb4f1ad3a3dd258131b0a366241bb2b4aa642b56
  • ghcr.io/openclarity/vmclarity-cli-dev:pr1149-eb4f1ad3a3dd258131b0a366241bb2b4aa642b56

@chrisgacsal chrisgacsal added this pull request to the merge queue Feb 1, 2024
Merged via the queue into main with commit 3f55f80 Feb 1, 2024
34 checks passed
@chrisgacsal chrisgacsal deleted the fix-make-test branch February 1, 2024 13:37
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants