Skip to content

Conversation

Parkreiner
Copy link
Member

@Parkreiner Parkreiner commented May 9, 2025

No issue to link – it's meant to address feedback from the previous README PR

Changes made

  • Updated all source properties in Terraform import snippets to use the new namespaced Terraform protocol URLs

Notes

  • Probably need to wait until the latest version of the Registry website is pushed to production before we merge this in, just to be on the safe side
  • I replaced all the paths via a regex, and then double-checked all the files modified to make sure there weren't any false positives

@Parkreiner Parkreiner self-assigned this May 9, 2025
@Parkreiner Parkreiner changed the title Mes/new readme files 2 chore: update all source URLs for modules to reflect new namespace format May 9, 2025
Copy link
Member

@matifali matifali left a comment

Choose a reason for hiding this comment

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

Thanks. Lets 🚢 it

@Parkreiner Parkreiner requested a review from bcpeinhardt May 12, 2025 17:18
@Parkreiner Parkreiner merged commit 84ce4ea into main May 13, 2025
3 checks passed
@Parkreiner Parkreiner deleted the mes/new-readme-files-2 branch May 13, 2025 21:53
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