This repository was archived by the owner on Apr 6, 2026. It is now read-only.
Implement dev_center_project_environment_type using GitHub Coding Agent with a new GH Issue Template#31
Merged
arnaudlh merged 3 commits intomicrosoft:mainfrom Oct 17, 2025
Merged
Conversation
…mplementing Terraform modeules.
…ter project-environment associations (#11) * Initial plan * Implement dev_center_project_environment_type module with tests and documentation Co-authored-by: raffertyuy <1037626+raffertyuy@users.noreply.github.com> * Complete dev_center_project_environment_type implementation with documentation updates Co-authored-by: raffertyuy <1037626+raffertyuy@users.noreply.github.com> * Fix issues encountered after a deployment of dev_center_project_environment_type. This includes fixing the naming convention of other modules (duplicate '-rg-rg-' in the prefix previously). * fix errors from unit tests * Fix API schema compliance for dev_center_project_environment_type module - Updated user role assignments schema to match Azure API requirements: - Changed roles from list(string) to map(object({})) - Modified example configurations to use user object IDs and role definition IDs - Enhanced README documentation with Azure CLI commands for ID retrieval - Files changed: - `modules/dev_center_project_environment_type/variables.tf`: Updated roles type definition - `modules/dev_center_project_environment_type/README.md`: Added advanced configuration details for user role assignments - `examples/dev_center_project_environment_type/enhanced_case/configuration.tfvars`: Updated user role assignments to use correct schema - `variables.tf`: Updated root variable definition to match module schema --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: raffertyuy <1037626+raffertyuy@users.noreply.github.com>
Contributor
Author
|
Added docs/vibe_coding.md with links to recorded videos. |
Contributor
Author
|
@microsoft-github-policy-service agree |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The key change in this PR is the new issue template in
.github/ISSUE_TEMPLATE/new_tf_module.mdwhich is pre-prompted so the coding agent will run/1-planand/2-implement.The idea is for the developer to test and run
/3-applymanually on PR code review. (FUTURE ENHANCEMENT is to create a GH Actions which test deploys to a shared sandbox azure subscription).With the above change, the following was also implemented as part of this PR
dev_center_project_environment_typeThis PR resolves #29