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

Removing deprecated AWS EC2 with Terraform and Digital Ocean App Platform V1 deployments #255

Merged
merged 5 commits into from
Sep 14, 2023

Conversation

scottisloud
Copy link
Collaborator

@scottisloud scottisloud commented Sep 14, 2023

This PR removes two deprecated deployment methods scheduled for deletion on 2023-09-14

  • AWS EC2 with Terraform
  • Digital Ocean App Platform V1

Both have been deprecated for longer than the documented period, and both have multiple viable alternatives.

As a bonus, this PR improves the appearance of a Note callout box.

@scottisloud scottisloud added the documentation For documentation-related issues label Sep 14, 2023
@scottisloud scottisloud self-assigned this Sep 14, 2023
deprecated/README.md Outdated Show resolved Hide resolved
Copy link
Member

@plttn plttn left a comment

Choose a reason for hiding this comment

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

one last minor tweak

deprecated/README.md Outdated Show resolved Hide resolved
Copy link
Member

@plttn plttn left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@ag-adampike ag-adampike left a comment

Choose a reason for hiding this comment

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

LGTM!

@scottisloud scottisloud merged commit 4b38853 into master Sep 14, 2023
@accraw accraw deleted the remove/aws-ec2-terraform-and-do-app-platform branch January 24, 2024 15:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation For documentation-related issues
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants