Skip to content

Conversation

@v-federicoar
Copy link
Contributor

Addressing
#122 (comment)

Removing Local Execution, integrating bicep files and instructions to deploy in Azure

@v-federicoar v-federicoar changed the title Removing Local Execution [Priortity queue] Removing Local Execution Dec 19, 2025
@skabou skabou requested a review from Copilot December 19, 2025 13:35
Copy link

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 removes local execution support for the Priority Queue pattern example and consolidates the Azure deployment infrastructure. The changes streamline the deployment process by integrating all Bicep templates into a single main template and removing local development instructions.

Key Changes:

  • Consolidated Azure Function App deployment from a separate Bicep file into the main deployment template
  • Removed local execution instructions and prerequisites from the documentation
  • Updated deployment commands to use the consolidated bicep/main.bicep file instead of a separate file

Reviewed changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated 2 comments.

File Description
priority-queue/bicep/sites.bicep New Bicep module that defines Function App infrastructure including app service plans, function apps, and role assignments
priority-queue/bicep/main.bicep Integrated storage account, Application Insights, and function app module deployment that was previously in a separate file
priority-queue/bicep/azure/azure-function-apps.bicep Deleted file - content consolidated into main.bicep
priority-queue/Readme.md Removed local execution instructions and updated to only show Azure deployment steps

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

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