Azure / terraform-azurerm-avm-template

Template repo for Azure Verified Modules using Terraform
MIT License
40 stars 27 forks source link

[AVM Question/Feedback]: clone jsonls under the Azure organization #109

Closed zioproto closed 4 months ago

zioproto commented 4 months ago

Check for previous/existing GitHub issues

Description

Since we depend on jsonls:

https://github.com/Azure/terraform-azurerm-avm-template/blob/d9fa53d6ecc9647d518afd2f1698f47fa2c8dd2c/.github/actions/e2e-getexamples/action.yml#L19C1-L20C1

Should the https://github.com/matt-FFFFFF/jsonls repo be moved under the Azure organization so that we can ensure proper governance of the code ?

thanks

microsoft-github-policy-service[bot] commented 4 months ago

[!IMPORTANT] The "Needs: Triage :mag:" label must be removed once the triage process is complete!

[!NOTE] This label was added as per ITA06.

microsoft-github-policy-service[bot] commented 4 months ago

[!WARNING] Tagging the AVM Core Team (@Azure/avm-core-team-technical-terraform) due to a module owner or contributor having not responded to this issue within 3 business days. The AVM Core Team will attempt to contact the module owners/contributors directly.

[!TIP]

  • To prevent further actions to take effect, the "Status: Response Overdue 🚩" label must be removed, once this issue has been responded to.
  • To avoid this rule being (re)triggered, the ""Needs: Triage :mag:" label must be removed as part of the triage process (when the issue is first responded to)!

[!NOTE] This message was posted as per ITA01TF.

matt-FFFFFF commented 4 months ago

Hi we can do, but this comes with its own issues.

Are there any other ways of easily getting a directory listing in a JSON array (with only dirs)?

matt-FFFFFF commented 4 months ago

Have proposed alternative in the referenced PR