Skip to content
This repository has been archived by the owner on Dec 16, 2021. It is now read-only.

ci: cache miss on release job #383

Merged
merged 1 commit into from
Jul 6, 2021
Merged

ci: cache miss on release job #383

merged 1 commit into from
Jul 6, 2021

Conversation

strophy
Copy link
Contributor

@strophy strophy commented Jul 6, 2021

Issue being fixed or feature implemented

Release builds were not hitting cache

What was done?

  • Use same cache key for release build

How Has This Been Tested?

In CI

Breaking Changes

None

Checklist:

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have added or updated relevant unit/integration/functional/e2e tests
  • I have made corresponding changes to the documentation

For repository code-owners and collaborators only

  • I have assigned this pull request to a milestone

@strophy strophy requested a review from shumkov July 6, 2021 13:24
@strophy strophy added this to the v0.20 milestone Jul 6, 2021
shumkov
shumkov approved these changes Jul 6, 2021
View reviewed changes
Copy link
Member

@shumkov shumkov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@shumkov shumkov changed the title fix: cache miss on release job ci: cache miss on release job Jul 6, 2021
@shumkov shumkov merged commit 4285a64 into v0.20-dev Jul 6, 2021
@shumkov shumkov deleted the fix-cache-key branch July 6, 2021 13:36
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@shumkov shumkov shumkov approved these changes

Assignees
No one assigned
Labels
None yet
Milestone
v0.20
Development

Successfully merging this pull request may close these issues.
2 participants
@strophy @shumkov