Skip to content

refactor: Refactor codebase by removing unused scripts and configurations#909

Merged
Avijit-Microsoft merged 7 commits into
devfrom
clean-unusedresources
May 13, 2026
Merged

refactor: Refactor codebase by removing unused scripts and configurations#909
Avijit-Microsoft merged 7 commits into
devfrom
clean-unusedresources

Conversation

@Kanchan-Microsoft
Copy link
Copy Markdown
Contributor

Purpose

This pull request updates documentation to reflect a migration from Azure Container Apps and Azure Container Registry to Azure App Service for hosting the application's web frontend and backend. It also updates related security guidance, quota and troubleshooting documentation, and deployment parameter descriptions. The most important changes are summarized below:

Resource and Deployment Documentation Updates:

  • Replaced references to Azure Container Apps and Azure Container Registry with Azure App Service in both the main README.md and documents/DeploymentGuide.md, including updated descriptions, pricing, and links. [1] [2]
  • Updated the list of customizable deployment parameters to remove container registry settings, reflecting the removal of Azure Container Registry as a required resource.

Security Guidance:

  • Updated security recommendations to use Virtual Network integration and access restrictions for Azure App Service, replacing previous guidance for Azure Container Apps.

Troubleshooting and Quota Documentation:

  • Removed troubleshooting entries related to Azure Container Apps limits and environment deletion, and added/updated guidance for Azure App Service capacity and related errors in documents/TroubleShootingSteps.md. [1] [2]

These updates ensure the documentation accurately reflects the new hosting architecture and provides relevant guidance for deployment, security, and troubleshooting.

Does this introduce a breaking change?

  • Yes
  • No

Golden Path Validation

  • I have tested the primary workflows (the "golden path") to ensure they function correctly without errors.

Deployment Validation

  • I have validated the deployment process successfully and all services are running as expected with this change.

What to Check

Verify that the following are valid

  • ...

Other Information

Copy link
Copy Markdown
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 pull request updates infrastructure and documentation to reflect a shift away from Azure Container Apps/ACR-focused deployment guidance and toward Azure App Service–based hosting, while removing legacy Container Apps IaC artifacts.

Changes:

  • Updated docs (README, DeploymentGuide, next-steps, troubleshooting) to reference Azure App Service and revise security/troubleshooting guidance accordingly.
  • Removed legacy Container Apps infrastructure templates/modules (infra/resources.bicep, infra/modules/fetch-container-image.bicep) and related naming data (infra/abbreviations.json).
  • Regenerated/updated compiled ARM template output (infra/main.json) and made small infra template refactors/comments.

Reviewed changes

Copilot reviewed 10 out of 11 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
src/App/package-lock.json Lockfile update (dependency tree entry removed).
README.md Resource list/security guidance updated to reference App Service.
next-steps.md Updated “what was added” + troubleshooting steps for App Service and main.bicep.
infra/resources.bicep Deleted legacy Container Apps + ACR-based resource deployment template.
infra/modules/fetch-container-image.bicep Deleted legacy helper for querying existing Container App images.
infra/main.json Updated generated ARM template (hash + variable/template content changes).
infra/main.bicep Comment/refactor tweaks and minor AI Foundry deployment gating simplification.
infra/main_custom.bicep Removed container image/ACR-related parameters; updated telemetry comments and outputs.
infra/abbreviations.json Deleted legacy abbreviations JSON used by the removed template.
documents/TroubleShootingSteps.md Removed Container Apps troubleshooting entries; added App Service capacity guidance and updated identity wording.
documents/DeploymentGuide.md Updated prerequisites and configurable-parameters text to remove Container Apps/ACR references.
Files not reviewed (1)
  • src/App/package-lock.json: Language not supported

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread README.md Outdated
Comment thread documents/DeploymentGuide.md Outdated
Comment thread infra/main.json
Co-authored-by: Copilot <copilot@github.com>
Copilot AI review requested due to automatic review settings May 13, 2026 06:01
Copy link
Copy Markdown
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

Copilot reviewed 10 out of 11 changed files in this pull request and generated 6 comments.

Files not reviewed (1)
  • src/App/package-lock.json: Language not supported

Comment thread README.md
Comment thread README.md
Comment thread documents/DeploymentGuide.md
Comment thread next-steps.md Outdated
Comment thread infra/main.json
Comment thread infra/main_custom.bicep
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings May 13, 2026 06:29
Copy link
Copy Markdown
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

Copilot reviewed 10 out of 11 changed files in this pull request and generated 4 comments.

Files not reviewed (1)
  • src/App/package-lock.json: Language not supported

Comment thread documents/DeploymentGuide.md
Comment thread infra/main_custom.bicep
Comment thread infra/main.json
Comment thread README.md
Copilot AI review requested due to automatic review settings May 13, 2026 08:23
Copy link
Copy Markdown
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

Copilot reviewed 10 out of 11 changed files in this pull request and generated 2 comments.

Files not reviewed (1)
  • src/App/package-lock.json: Language not supported

Comment thread README.md
Comment thread documents/DeploymentGuide.md
@Avijit-Microsoft Avijit-Microsoft merged commit 9bcc8e1 into dev May 13, 2026
9 checks passed
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.

3 participants