Skip to content

Conversation

@matebarabas
Copy link
Contributor

Pull Request Checklist

  • I have read and followed the CONTRIBUTING.md guidelines.
  • My contribution adds a new instruction, prompt, or chat mode file in the correct directory.
  • The file follows the required naming convention.
  • The content is clearly structured and follows the example format.
  • I have tested my instructions, prompt, or chat mode with GitHub Copilot.
  • I have run npm start and verified that README.md is up to date.

Description


Type of Contribution

  • New instruction file.
  • New prompt file.
  • New chat mode file.
  • New collection file.
  • Update to existing instruction, prompt, chat mode, or collection.
  • Other (please specify):

Additional Notes


By submitting this pull request, I confirm that my contribution abides by the Code of Conduct and will be licensed under the MIT License.

Copilot AI review requested due to automatic review settings November 11, 2025 01:51
Copilot finished reviewing on behalf of matebarabas November 11, 2025 01:52
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the Azure Verified Modules (AVM) Terraform instruction file to reflect changes in the official module index links. The links have been migrated from HTML pages to CSV files hosted on GitHub.

Key changes:

  • Updated Terraform module index links from HTML pages to raw GitHub CSV files
  • Added new "Terraform Utility Modules" category
  • Removed Bicep-related links that were not relevant to the Terraform-specific instruction file

- Example: Search "avm storage account" → filter by Partner

### Official AVM Index

Copy link

Copilot AI Nov 11, 2025

Choose a reason for hiding this comment

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

[nitpick] The links use refs/heads/main which points to the latest commit on the main branch. This could lead to inconsistent documentation if the CSV files are moved or renamed. Consider using a stable permalink or version tag, or document that these links are intentionally tracking the latest version.

Suggested change
> **Note:** The following links use `refs/heads/main` to always point to the latest version of the CSV files on the main branch. This means the files may change over time, and their location or content may be updated. If you require a stable version, consider using a specific release tag in the URL.

Copilot uses AI. Check for mistakes.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Added a more applicable note.

@matebarabas
Copy link
Contributor Author

@PlagueHO, @segraef, I've updated the links for this TF specific instructions file.

As recommended by Copilot review, added a note regarding the links to the latest CSV files and suggested using specific release tags for point-in-time versions.
@aaronpowell aaronpowell merged commit de74216 into github:main Nov 11, 2025
2 checks passed
@matebarabas matebarabas deleted the patch-1 branch November 11, 2025 04:35
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