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.

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

(CONT-257) Add PDKTemplateRef parameter #231

Merged
merged 2 commits into from
Nov 3, 2022

Conversation

chelnak
Copy link

@chelnak chelnak commented Nov 3, 2022

Prior to this commit it was not possible to specify the version of pdk-templates that would be used when puppetizing a Dsc module.

PDK help sugegsts that the default ref is 2.5.0, however in practice it would pull down the latest tag from the pdk-templates repository. At the time of commit, this was 2.6.0.

The above diff causes some issues when puppetizing a resource due to gem incompatibilities.

This commit adds a new parameter called PDKTemplateRef to New-PuppetDscModule and Initialize-PuppetModule so that users can optionally provide a different pdk-templates version.

The default value has been set to 2.5.0.

@chelnak chelnak added the bug Something isn't working label Nov 3, 2022
@chelnak chelnak requested a review from a team as a code owner November 3, 2022 14:07
@chelnak chelnak self-assigned this Nov 3, 2022
@chelnak chelnak force-pushed the CONT-257-add_pdk_template_ref_param branch from 3ce9862 to f062a03 Compare November 3, 2022 14:13
Prior to this commit it was not possible to specify the version of
pdk-templates that would be used when puppetizing a Dsc module.

PDK help sugegsts that the default ref is 2.5.0, however in practice it
would pull down the latest tag from the pdk-templates repository. At the
time of commit, this was 2.6.0.

The above diff causes some issues when puppetizing a resource due to gem
incompatibilities.

This commit adds a new parameter called PDKTemplateRef to
`New-PuppetDscModule` and `Initialize-PuppetModule` so that users can
optionally provide a different pdk-templates version.

The default value has been set to 2.5.0.
@chelnak chelnak force-pushed the CONT-257-add_pdk_template_ref_param branch from f062a03 to 543ec44 Compare November 3, 2022 14:23
@chelnak chelnak closed this Nov 3, 2022
@chelnak chelnak reopened this Nov 3, 2022
Copy link

@GSPatton GSPatton left a comment

Choose a reason for hiding this comment

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

This looks good to me. Powershell scripts are surprisingly pleasant to read

@GSPatton GSPatton closed this Nov 3, 2022
@GSPatton
Copy link

GSPatton commented Nov 3, 2022

rerunning stuck tests

@GSPatton GSPatton reopened this Nov 3, 2022
@GSPatton
Copy link

GSPatton commented Nov 3, 2022

2016 tests failing due to the runner being deprecated

2016 runners have now been removed.

This commit adds 2022 runners and removes 2016 runners.
@GSPatton GSPatton merged commit eb47b62 into main Nov 3, 2022
@GSPatton GSPatton deleted the CONT-257-add_pdk_template_ref_param branch November 3, 2022 17:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants