NotificationsYou must be signed in to change notification settings
Fork45
Star349

Add`if()` function#776

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.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

Merged

SteveL-MSFT merged 2 commits intoPowerShell:mainfromSteveL-MSFT:if-function

May 1, 2025

Merged

Add`if()` function#776

SteveL-MSFT merged 2 commits intoPowerShell:mainfromSteveL-MSFT:if-function

May 1, 2025

Conversation

Copy link

Member

SteveL-MSFT commentedMay 1, 2025

PR Summary

Add theif() function needed for Bicep support

PR Context

Part of#767

SteveL-MSFT requested review fromandyleejordan andtgauth

May 1, 2025 21:33

Addif() function

1781064

SteveL-MSFT force-pushed theif-function branch from48e0e33 to1781064Compare

May 1, 2025 21:35

missed a merge conflict

cec2387

andyleejordan approved these changes

May 1, 2025

View reviewed changes

Copy link

Member

andyleejordan left a comment

Choose a reason for hiding this comment

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

Amazing, this now works:

Q:\src\DSC\bin\debug\dsc.exe -l debug config --parameters '{"parameters": {"colorMode": "light"}}' set --file .\demo.json

with this ternary in Bicep:

@allowed(['light', 'dark'])@description('Light or dark color mode')// TODO: Teach DSC how to use Bicep param filesparam colorMode string = 'light'var valueData = colorMode == 'light' ? 1 : 0var keyPathPrefix = 'HKCU\\SOFTWARE\\Microsoft\\Windows\\CurrentVersion'resource systemUsesLightTheme 'Microsoft.Windows/Registry@2025-04-07' = {    name: 'SystemUsesLightTheme'    properties: {        keyPath: concat(keyPathPrefix, '\\Themes\\Personalize')        valueName: 'SystemUsesLightTheme'        valueData: {          DWord: valueData        }    }}