Skip to content

fix: BCPNFR1 update to include RDT and remove CARML note#2132

Merged
eriqua merged 2 commits intomainfrom
users/erikag/udt-rdt
Jun 20, 2025
Merged

fix: BCPNFR1 update to include RDT and remove CARML note#2132
eriqua merged 2 commits intomainfrom
users/erikag/udt-rdt

Conversation

@eriqua
Copy link
Contributor

@eriqua eriqua commented Jun 13, 2025

Overview/Summary

image

This PR fixes/adds/changes/removes

  1. Replace me
  2. Replace me
  3. Replace me

Breaking Changes

  1. Replace me
  2. Replace me

As part of this Pull Request I have

  • Read the Contribution Guide and ensured this PR is compliant with the guide
  • Checked for duplicate Pull Requests
  • Associated it with relevant GitHub Issues or ADO Work Items (Internal Only)
  • Ensured my code/branch is up-to-date with the latest changes in the main branch
  • Ensured PR tests are passing
  • Updated relevant and associated documentation (e.g. Contribution Guide, Docs etc.)

@eriqua eriqua self-assigned this Jun 13, 2025
@microsoft-github-policy-service microsoft-github-policy-service bot added the Needs: Triage 🔍 Maintainers need to triage still label Jun 13, 2025
@eriqua eriqua marked this pull request as ready for review June 19, 2025 16:18
@eriqua eriqua requested a review from a team as a code owner June 19, 2025 16:18
@eriqua eriqua enabled auto-merge (squash) June 19, 2025 16:19
@AlexanderSehr
Copy link
Contributor

Is the "complex types" terminology something official? Just asking as it's the first time I hear about it 🙂

@eriqua
Copy link
Contributor Author

eriqua commented Jun 19, 2025

Is the "complex types" terminology something official? Just asking as it's the first time I hear about it 🙂

I'd say more a general programming language term than IaC specific, as a type composed of other types. We use it in other places as well, e.g., SNFR14 and also in the existing version of this same spec.

That said, if you have a suggestion, shoot 🔫

@AlexanderSehr
Copy link
Contributor

Is the "complex types" terminology something official? Just asking as it's the first time I hear about it 🙂

I'd say more a general programming language term than IaC specific, as a type composed of other types. We use it in other places as well, e.g., SNFR14 and also in the existing version of this same spec.

That said, if you have a suggestion, shoot 🔫

Nah, all good. Was just curious as we did not use it before and I thought it's something specific chosen to represent UDTs & RTS.

@eriqua eriqua merged commit f79f207 into main Jun 20, 2025
7 checks passed
@eriqua eriqua deleted the users/erikag/udt-rdt branch June 20, 2025 07:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Needs: Triage 🔍 Maintainers need to triage still

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants