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

feat: add cloudfront cache policy resource #136

Merged
merged 3 commits into from
Apr 13, 2024

Conversation

ekristen
Copy link
Owner

@ekristen ekristen commented Apr 13, 2024

Overview

This adds the cloudfront cache policy by way of cherry picked commit from @MikeSchouw and their upstream branch based on this PR rebuy-de/aws-nuke#1152

Rebased the original commit to turn the commit into semantic commit message format, but preserve Mike's contributions as much as possible. Converted to the libnuke resource format and add a nolint statement for golangci-lint to ignore.

Resolves #135

Upstream References

@ekristen ekristen merged commit 0e99104 into main Apr 13, 2024
12 checks passed
@ekristen ekristen deleted the feature-cloudfront-cache-policy branch April 13, 2024 01:08
@ekristen
Copy link
Owner Author

🎉 This PR is included in version 3.0.0-beta.40 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@Petersoj
Copy link
Contributor

Love the cherry pick to preserve contribution credit!

@ekristen
Copy link
Owner Author

Thanks. Trying to keep credit where credit is due when I'm not the primary repo.

@ekristen
Copy link
Owner Author

🎉 This PR is included in version 3.0.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Merge Upstream PRs
3 participants