Skip to content

🔨refactor: Update GitHub Actions prompt mode and add Azure Storage Account configuration #62

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

0GiS0
Copy link
Owner

@0GiS0 0GiS0 commented May 28, 2025

GitHub Actions Improvements:

  • Updated the .github/prompts/github-actions.prompt.md file to change the mode from 'ask' to 'agent', aligning with a more automated approach for GitHub Actions optimizations.

Terraform Configuration Enhancements:

  • New Azure Storage Account Module:
    • Added a new module "storage_account" in terraform/main.tf to provision an Azure Storage Account, including variables for name, resource group, location, and tags.
    • Introduced corresponding outputs in terraform/outputs.tf for storage_account_name and storage_account_id to expose the Storage Account's details.
  • New Variable for Storage Account:
    • Added a variable "storage_account_name" in terraform/variables.tf with a default value that incorporates a random integer suffix for uniqueness.
    • Updated terraform/terraform.tfvars.example to include an example value for storage_account_name.

@0GiS0 0GiS0 requested a review from Copilot May 28, 2025 10:53
Copy link

@Copilot 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 GitHub Actions prompt configuration for more automated suggestions and adds Terraform support for provisioning an Azure Storage Account.

  • Switches the GitHub Actions prompt mode from ask to agent
  • Introduces a new Terraform module to create an Azure Storage Account
  • Adds storage_account_name variable, example value, and outputs for the new Storage Account

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

Show a summary per file
File Description
.github/prompts/github-actions.prompt.md Change prompt mode to agent
terraform/variables.tf Add storage_account_name variable with default
terraform/terraform.tfvars.example Provide example storage_account_name
terraform/main.tf Add storage_account module configuration
terraform/outputs.tf Expose Storage Account name and ID via outputs
Comments suppressed due to low confidence (3)

terraform/variables.tf:115

  • Ensure that the random_integer.suffix resource is declared elsewhere in the configuration; without it, this default reference will cause a Terraform plan error.
default     = "heroesstorage${random_integer.suffix.result}"

terraform/main.tf:66

  • The provider map has a duplicate key (azurerm.this). Remove or correct this entry to avoid map literal errors in Terraform.
    azurerm.this = azurerm.this

terraform/main.tf:73

  • [nitpick] Consider translating this comment to English to maintain consistency in codebase language.
# Puedes añadir más variables según la documentación del módulo

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.

1 participant