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

Back out "feat(caches): support opting into or out of specific name c… #627

Merged
merged 1 commit into from
Jan 15, 2025

Conversation

ahal
Copy link
Collaborator

@ahal ahal commented Jan 15, 2025

…aches"

This backs out commit a70a57c.

@ahal ahal self-assigned this Jan 15, 2025
@ahal
Copy link
Collaborator Author

ahal commented Jan 15, 2025

For context I should have held off landing this as I ended up deciding that it was better to have well defined caches that need to be opted into explicitly.

I want to avoid having a release with this regex feature to ensure we don't have users accidentally using it and then needing to stop using it on the next release.

Copy link
Contributor

@jcristau jcristau left a comment

Choose a reason for hiding this comment

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

nit: would love some explanation for the revert as part of the commit message :)

@ahal ahal enabled auto-merge (rebase) January 15, 2025 15:15
@ahal ahal disabled auto-merge January 15, 2025 15:15
…aches"

This backs out commit a70a57c.

For context I should have held off landing this as I ended up deciding
that it was better to have well defined caches that need to be opted
into explicitly.

I want to avoid having a release with this regex feature to ensure we
don't have users accidentally using it and then needing to stop using it
on the next release.
@ahal ahal force-pushed the ahal/push-romwuotwlvlm branch from 5d4f050 to 7b18977 Compare January 15, 2025 15:15
@ahal ahal merged commit 15ff22c into taskcluster:main Jan 15, 2025
16 checks passed
@ahal ahal deleted the ahal/push-romwuotwlvlm branch January 15, 2025 15:49
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.

2 participants