Skip to content
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.

Sign up for GitHub

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

Jump to bottom

feat: new module avm-res-network-expressrouteport #4259

Merged
merged 87 commits into from
Jan 27, 2025
Merged

feat: new module avm-res-network-expressrouteport #4259

merged 87 commits into from
Jan 27, 2025

Conversation

ChrisSidebotham
Copy link
Contributor

Description

Pipeline Reference

Pipeline
avm.res.network.express-route-port

Type of Change

  • Update to CI Environment or utilities (Non-module affecting changes)
  • Azure Verified Module updates:
    • Bugfix containing backwards-compatible bug fixes, and I have NOT bumped the MAJOR or MINOR version in version.json:
      • Someone has opened a bug report issue, and I have included "Closes #{bug_report_issue_number}" in the PR description.
      • The bug was found by the module author, and no one has opened an issue to report it yet.
    • Feature update backwards compatible feature updates, and I have bumped the MINOR version in version.json.
    • Breaking changes and I have bumped the MAJOR version in version.json.
    • Update to documentation

Checklist

  • I'm sure there are no other open Pull Requests for the same update/change
  • I have run Set-AVMModule locally to generate the supporting module files.
  • My corresponding pipelines / checks run clean and green without any errors or warnings

ChrisSidebotham and others added 30 commits October 12, 2023 14:59
@ChrisSidebotham
adding deployment files
7ecd1a5
@ChrisSidebotham
ficing rp ref
0f15c86
@ChrisSidebotham
Merge branch 'main' of https://github.com/Azure/bicep-registry-modules
c6993a6
@ChrisSidebotham
Merge branch 'main' of https://github.com/Azure/bicep-registry-modules
0e65d72
@ChrisSidebotham
Adding workflow file
482add6
@ChrisSidebotham
Merge branch 'Azure:main' into main
1bffa61
@ChrisSidebotham
Merge branch 'main' of https://github.com/Azure/bicep-registry-modules
8c358e7
@ChrisSidebotham
Merge branch 'main' of https://github.com/ChrisSidebotham/bicep-regis…
14a5d0d 
…try-modules
@ChrisSidebotham
fixing path
fba5375
@ChrisSidebotham
Fixing Workflow name
05152a8
@ChrisSidebotham
adding workflow
f8d04c6
@ChrisSidebotham
Merge branch 'main' of https://github.com/Azure/bicep-registry-modules
d22a6c7
@ChrisSidebotham
Adding Workflow file
6993c4d
@ChrisSidebotham
Adding workflow file
cf44304
@ChrisSidebotham
Merge branch 'anchor' of https://github.com/ChrisSidebotham/bicep-reg…
5260056 
…istry-modules
@ChrisSidebotham
Merge branch 'main' of https://github.com/Azure/bicep-registry-modules
3740075
@ChrisSidebotham
Adding workflow file
c9b6b4c
@ChrisSidebotham
adding workflow
a3d83a4
@ChrisSidebotham
Fixing workflow name
915d12c
@ChrisSidebotham
update workflow
1c56bb3
@ChrisSidebotham
Merge branch 'main' of https://github.com/Azure/bicep-registry-modules
454bcc5
@ChrisSidebotham
updating Test Case s location to fix Pipeline
fe017c1
@ChrisSidebotham
Merge branch 'main' of https://github.com/Azure/bicep-registry-modules
ddb6e04
@ChrisSidebotham
Merge branch 'main' of https://github.com/Azure/bicep-registry-modules
1931d9a
@ChrisSidebotham
Merge branch 'main' of https://github.com/Azure/bicep-registry-modules …
657311c 
…into anchor
@ChrisSidebotham
Merge branch 'main' of https://github.com/Azure/bicep-registry-modules
fff3279
@ChrisSidebotham
Merge branch 'main' of https://github.com/Azure/bicep-registry-modules …
c5f6e4f 
…into anchor
@ChrisSidebotham
Merge branch 'main' of https://github.com/Azure/bicep-registry-modules
ffc5837
@ChrisSidebotham
adding workflow
e9b2e80
@ChrisSidebotham
Merge branch 'main' of https://github.com/Azure/bicep-registry-modules …
018c016 
…into anchor
@eriqua eriqua added the Status: Do Not Merge ⛔ Do not merge PRs with this label attached as they are not ready or aligned to future direction etc. label Jan 21, 2025
@eriqua eriqua added Class: Resource Module 📦 This is a resource module and removed Status: Do Not Merge ⛔ Do not merge PRs with this label attached as they are not ready or aligned to future direction etc. Needs: Triage 🔍 Maintainers need to triage still labels Jan 25, 2025
arnoldna
arnoldna previously approved these changes Jan 27, 2025
View reviewed changes
eriqua
eriqua approved these changes Jan 27, 2025
View reviewed changes
Copy link
Contributor

@eriqua eriqua left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🆗

@ChrisSidebotham ChrisSidebotham merged commit ccc4bbe into Azure:main Jan 27, 2025
4 checks passed
@ChrisSidebotham ChrisSidebotham deleted the express-route-ports branch January 27, 2025 17:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@eriqua eriqua eriqua approved these changes

@arnoldna arnoldna arnoldna left review comments

@AlexanderSehr AlexanderSehr Awaiting requested review from AlexanderSehr

Assignees

@AlexanderSehr AlexanderSehr

@ChrisSidebotham ChrisSidebotham

@eriqua eriqua

Labels
Class: Resource Module 📦 This is a resource module Needs: Core Team 🧞 This item needs the AVM Core Team to review it Type: AVM 🅰️ ✌️ Ⓜ️ This is an AVM related issue
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@ChrisSidebotham @AlexanderSehr @arnoldna @eriqua