Skip to content

Add the Sub-Modules as RequiredModules in the Root Module Manifest #1194

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
Merged
emmanuel-karanja merged 7 commits into from
Nov 14, 2024
Merged

Add the Sub-Modules as RequiredModules in the Root Module Manifest #1194

emmanuel-karanja merged 7 commits into from
Nov 14, 2024

Conversation

@emmanuel-karanja
Copy link
Contributor

@emmanuel-karanja emmanuel-karanja commented Nov 8, 2024

Add the Sub-Modules as RequiredModules in the Root Module Manifest

@emmanuel-karanja emmanuel-karanja requested a review from a team as a code owner November 8, 2024 05:36
@learn-build-service-prod
Copy link

learn-build-service-prod bot commented Nov 8, 2024

Learn Build status updates of commit ee314e8:

✅ Validation status: passed

File Status Preview URL Details
build/Beta-TypeDefs.txt ✅Succeeded
build/common-functions.ps1 ✅Succeeded
build/Create-EntraModule.ps1 ✅Succeeded
build/Create-ModuleMapping.ps1 ✅Succeeded
build/Split-Docs.ps1 ✅Succeeded
build/Split-EntraModule.ps1 ✅Succeeded
build/Split-Tests.ps1 ✅Succeeded
build/Update-CommonFunctionsImport.ps1 ✅Succeeded
build/V1.0-TypeDefs.txt ✅Succeeded
build/VNext-Build.md ✅Succeeded
module/docs/entra-powershell-v1.0/Microsoft.Graph.Entra/Set-EntraUserExtension.md ✅Succeeded View (entra-powershell-preview)
module/Entra/config/dependencyMapping.json ✅Succeeded
module/Entra/config/moduleMapping.json ✅Succeeded
module/Entra/config/ModuleSettings.json ✅Succeeded
module/Entra/customizations/Set-EntraUserExtension.ps1 ✅Succeeded
module/EntraBeta/config/dependencyMapping.json ✅Succeeded
module/EntraBeta/config/moduleMapping.json ✅Succeeded
module/EntraBeta/config/ModuleSettings.json ✅Succeeded
moduleVNext/Entra/config/dependencyMapping.json ✅Succeeded
moduleVNext/Entra/config/moduleMapping.json ✅Succeeded
moduleVNext/Entra/config/ModuleMetadata.json ✅Succeeded
moduleVNext/Entra/config/ModuleSettings.json ✅Succeeded
moduleVNext/Entra/Microsoft.Graph.Entra/Applications/Add-EntraApplicationOwner.ps1 ✅Succeeded
moduleVNext/Entra/Microsoft.Graph.Entra/Applications/Add-EntraServicePrincipalDelegatedPermissionClassification.ps1 ✅Succeeded
moduleVNext/Entra/Microsoft.Graph.Entra/Applications/Add-EntraServicePrincipalOwner.ps1 ✅Succeeded

This comment lists only the first 25 files in the pull request.
For more details, please refer to the build report.

For any questions, please:

@emmanuel-karanja emmanuel-karanja changed the base branch from main to modularize November 8, 2024 05:38
@learn-build-service-prod
Copy link

learn-build-service-prod bot commented Nov 8, 2024

Learn Build status updates of commit 14b58cc:

✅ Validation status: passed

File Status Preview URL Details
build/Beta-TypeDefs.txt ✅Succeeded
build/common-functions.ps1 ✅Succeeded
build/Create-EntraModule.ps1 ✅Succeeded
build/Create-ModuleMapping.ps1 ✅Succeeded
build/Split-Docs.ps1 ✅Succeeded
build/Split-EntraModule.ps1 ✅Succeeded
build/Split-Tests.ps1 ✅Succeeded
build/Update-CommonFunctionsImport.ps1 ✅Succeeded
build/V1.0-TypeDefs.txt ✅Succeeded
build/VNext-Build.md ✅Succeeded
module/docs/entra-powershell-v1.0/Microsoft.Graph.Entra/Set-EntraUserExtension.md ✅Succeeded View (entra-powershell-preview)
module/Entra/config/dependencyMapping.json ✅Succeeded
module/Entra/config/moduleMapping.json ✅Succeeded
module/Entra/config/ModuleSettings.json ✅Succeeded
module/Entra/customizations/Set-EntraUserExtension.ps1 ✅Succeeded
module/EntraBeta/config/dependencyMapping.json ✅Succeeded
module/EntraBeta/config/moduleMapping.json ✅Succeeded
module/EntraBeta/config/ModuleSettings.json ✅Succeeded
moduleVNext/Entra/config/dependencyMapping.json ✅Succeeded
moduleVNext/Entra/config/moduleMapping.json ✅Succeeded
moduleVNext/Entra/config/ModuleMetadata.json ✅Succeeded
moduleVNext/Entra/config/ModuleSettings.json ✅Succeeded
moduleVNext/Entra/Microsoft.Graph.Entra/Applications/Add-EntraApplicationOwner.ps1 ✅Succeeded
moduleVNext/Entra/Microsoft.Graph.Entra/Applications/Add-EntraServicePrincipalDelegatedPermissionClassification.ps1 ✅Succeeded
moduleVNext/Entra/Microsoft.Graph.Entra/Applications/Add-EntraServicePrincipalOwner.ps1 ✅Succeeded

This comment lists only the first 25 files in the pull request.
For more details, please refer to the build report.

For any questions, please:

@learn-build-service-prod
Copy link

learn-build-service-prod bot commented Nov 8, 2024

Learn Build status updates of commit 14b58cc:

✅ Validation status: passed

File Status Preview URL Details
build/Create-EntraModule.ps1 ✅Succeeded
src/EntraModuleBuilder.ps1 ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

@emmanuel-karanja emmanuel-karanja merged commit 5f9291d into modularize Nov 14, 2024
2 checks passed
@learn-build-service-prod
Copy link

learn-build-service-prod bot commented Nov 14, 2024

Learn Build status updates of commit 126cd4d:

✅ Validation status: passed

File Status Preview URL Details
build/Create-EntraModule.ps1 ✅Succeeded
src/EntraModuleBuilder.ps1 ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

KenitoInc pushed a commit that referenced this pull request Nov 21, 2024
@emmanuel-karanja @KenitoInc
Add the Sub-Modules as RequiredModules in the Root Module Manifest (#…
a91f137 
…1194)

* Sub-Modules as RequiredModules

* Sub-Modules as RequiredModules

* Sub-Modules as RequiredModules

* Sub-Modules as RequiredModules

* Sub-Modules as RequiredModules

* Sub-Modules as RequiredModules
KenitoInc pushed a commit that referenced this pull request Jan 31, 2025
@emmanuel-karanja @KenitoInc
Add the Sub-Modules as RequiredModules in the Root Module Manifest (#…
f65f649 
…1194)

* Sub-Modules as RequiredModules

* Sub-Modules as RequiredModules

* Sub-Modules as RequiredModules

* Sub-Modules as RequiredModules

* Sub-Modules as RequiredModules

* Sub-Modules as RequiredModules
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@KenitoInc KenitoInc Awaiting requested review from KenitoInc

@SteveMutungi254 SteveMutungi254 Awaiting requested review from SteveMutungi254

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@emmanuel-karanja