Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

chore: Remove terratest workflow and correct documentation links #1441

Merged
merged 3 commits into from
Feb 17, 2023

Conversation

bryantbiggs
Copy link
Contributor

@bryantbiggs bryantbiggs commented Feb 17, 2023

What does this PR do?

  • Remove terratest workflow; we can re-evaluate in the future based on what our standardized testing strategy will be for patterns
  • Correct broken links in documentation
  • Update documentation to reflect changes in v5

Motivation

More

  • Yes, I have tested the PR using my local account setup (Provide any test evidence report under Additional Notes)
  • Yes, I have updated the docs for this feature
  • Yes, I ran pre-commit run -a with this PR

For Moderators

  • E2E Test successfully complete before merge?

Additional Notes

@bryantbiggs bryantbiggs requested a review from a team as a code owner February 17, 2023 00:49
@bryantbiggs bryantbiggs temporarily deployed to EKS Blueprints Test February 17, 2023 00:49 — with GitHub Actions Inactive
@bryantbiggs bryantbiggs temporarily deployed to EKS Blueprints Test February 17, 2023 00:53 — with GitHub Actions Inactive
@bryantbiggs bryantbiggs merged commit db938bc into main Feb 17, 2023
@bryantbiggs bryantbiggs deleted the chore/remove-terratest branch February 17, 2023 11:51
dhoeld pushed a commit to dhoeld/terraform-aws-eks-blueprints that referenced this pull request Feb 24, 2023
gminiba pushed a commit to gminiba/terraform-aws-eks-blueprints that referenced this pull request Mar 17, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants