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

Use IC_VERSION only on tags in CI #2382

Merged
merged 1 commit into from Jan 25, 2022
Merged

Use IC_VERSION only on tags in CI #2382

merged 1 commit into from Jan 25, 2022

Conversation

lucacome
Copy link
Member

Only use the version on new tags, that's when we actually only need that value to be set.

@lucacome lucacome requested a review from a team January 24, 2022 21:04
@lucacome lucacome self-assigned this Jan 24, 2022
@lucacome lucacome requested review from pleshakov and ciarams87 and removed request for a team January 24, 2022 21:04
@github-actions github-actions bot added the chore Pull requests for routine tasks label Jan 24, 2022
@codecov-commenter
Copy link

Codecov Report

Merging #2382 (3a73656) into master (9d393ba) will decrease coverage by 0.02%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2382      +/-   ##
==========================================
- Coverage   53.69%   53.66%   -0.03%     
==========================================
  Files          48       48              
  Lines       14201    14201              
==========================================
- Hits         7625     7621       -4     
- Misses       6338     6340       +2     
- Partials      238      240       +2     
Impacted Files Coverage Δ
internal/k8s/configuration.go 95.47% <0.00%> (-0.39%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9d393ba...3a73656. Read the comment docs.

@lucacome lucacome changed the title Use IC_VERSION only on tags Use IC_VERSION only on tags in CI Jan 25, 2022
@lucacome lucacome merged commit 88a255b into master Jan 25, 2022
@lucacome lucacome deleted the chore/image-cache branch January 25, 2022 21:46
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
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants