Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Only add the test container to cache once #3518

Merged
merged 1 commit into from Feb 7, 2023
Merged

Conversation

lucacome
Copy link
Member

@lucacome lucacome commented Feb 3, 2023

  • Builds the container and adds it to cache once before starting all the tests
  • Skips running the tests on tag

@lucacome lucacome requested a review from a team as a code owner February 3, 2023 07:59
@lucacome lucacome self-assigned this Feb 3, 2023
@github-actions github-actions bot added the chore Pull requests for routine tasks label Feb 3, 2023
@codecov-commenter
Copy link

codecov-commenter commented Feb 3, 2023

Codecov Report

Merging #3518 (fb618ea) into main (879efeb) will decrease coverage by 0.03%.
The diff coverage is n/a.

❗ Current head fb618ea differs from pull request most recent head deaf037. Consider uploading reports for the commit deaf037 to get more accurate results

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@            Coverage Diff             @@
##             main    #3518      +/-   ##
==========================================
- Coverage   52.00%   51.98%   -0.03%     
==========================================
  Files          60       60              
  Lines       16816    16816              
==========================================
- Hits         8746     8742       -4     
- Misses       7775     7777       +2     
- Partials      295      297       +2     
Impacted Files Coverage Δ
internal/k8s/configuration.go 95.43% <0.00%> (-0.37%) ⬇️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@lucacome lucacome merged commit 2efa8d4 into main Feb 7, 2023
@lucacome lucacome deleted the chore/test-container branch February 7, 2023 05:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Pull requests for routine tasks
Projects
Status: Done 🚀
Development

Successfully merging this pull request may close these issues.

None yet

4 participants