-
Notifications
You must be signed in to change notification settings - Fork 93
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
[Feedback]: Implement automated checks for the module index CSVs #382
Comments
Thanks Mate - Will review next week and maybe get some time with @AlexanderSehr |
I hope Regex is strong with you my friend :D |
GitHub Copilot Chat is the name of the game when it comes to regex! ;) That is if they'll be needed at all, as most things we'll need can be translated to splitting/concatenating strings and replacing hyphens with forward slashes or back. :) |
@matebarabas & @ChrisSidebotham guys let us know if there is anything for terraform related index that we need to discuss for this |
I think the first iteration of this solution should be fairly generic. We'd then need to add additional checks later, and this is where we might have differences between Bicep and TF, e.g. naming conventions for patterns. These will just naturally come up, and we'll discuss them along the way! ;) |
@ChrisSidebotham, I've consolidated #890 into this issue as it was redundant. Let's have a chat internally to decide if you could help with this. Thanks! |
Check for previous/existing GitHub issues
Description
Implement a GitHub workflow that validates the module index CSVs during PRs: : check all records in all 4 module index CSV files (Bicep/TF, res/ptn modules).
Scenarios to test include but aren't limited to:
Proposed :new:
Available :green_circle:
Orphaned :eyes:
Available :green_circle:
avm/res/
oravm-res
MUST be equal to theProviderNamespace
's value (first column of the resource CSVs) - i.e., no conversion to singular form!@ChrisSidebotham - could you please help with this? Thanks!
CC: @AlexanderSehr
The text was updated successfully, but these errors were encountered: