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

refactor: remove compile-built-releases #306

Merged
merged 2 commits into from
Feb 22, 2022

Conversation

crhntr
Copy link
Contributor

@crhntr crhntr commented Jan 14, 2022

Similar functionality is now handled by cache-compiled-releases. This is no longer used in our pipelines so we should remove it.

@cf-gitbot
Copy link
Member

We have created an issue in Pivotal Tracker to manage this. Unfortunately, the Pivotal Tracker project is private so you may be unable to view the contents of the story.

The labels on this github issue will be updated when the story is started.

@crhntr crhntr force-pushed the refactor/remove-compile-built-releases branch from a224b94 to 5e067ee Compare January 14, 2022 00:46
@crhntr crhntr changed the title [refactor]: remove command and most associated code [refactor]: remove compile-built-releases and most associated code Jan 14, 2022
@crhntr crhntr changed the title [refactor]: remove compile-built-releases and most associated code [refactor]: remove compile-built-releases Jan 14, 2022
@crhntr crhntr changed the title [refactor]: remove compile-built-releases refactor: remove compile-built-releases Jan 14, 2022
@crhntr crhntr marked this pull request as ready for review January 14, 2022 20:10
@crhntr crhntr force-pushed the refactor/remove-compile-built-releases branch from 0d6347f to 5e067ee Compare January 14, 2022 20:28
Copy link
Contributor

@dtimm dtimm left a comment

Choose a reason for hiding this comment

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

LGTM

@dtimm dtimm merged commit 101ec59 into main Feb 22, 2022
@dtimm dtimm deleted the refactor/remove-compile-built-releases branch February 22, 2022 22:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants