Skip to content

Conversation

@anmenaga
Copy link

@anmenaga anmenaga commented Jun 7, 2021

PR Summary

Currently PSDesiredStateConfiguration v2.0.5 module ships inbox with PS releases.
This module is now published on PS Gallery and should be installed from PS Gallery when necessary by something like Install-Module -Name PSDesiredStateConfiguration -Repository PSGallery -MaximumVersion 2.99.
This PR removes v2.0.5 module from being shipped in future PS releases.

Context: Announcing PSDesiredStateConfiguration on PowerShell Gallery.

Special case worth mentioning: this PR removes PSDesiredStateConfiguration.InvokeDscResource experimental feature from being enabled by default in future PS Previews (because the module will no longer be shipped with them). This means that users who need that functionality will have to enable this experimental feature after downloading the module from PS Gallery.

PR Checklist

@anmenaga anmenaga added the CL-BuildPackaging Indicates that a PR should be marked as a build or packaging change in the Change Log label Jun 7, 2021
@ghost ghost assigned rjmholt Jun 7, 2021
# Make sure ExperimentalFeatures from modules in PSHome are added
# https://github.com/PowerShell/PowerShell/issues/10550
@("PSDesiredStateConfiguration.InvokeDscResource") | ForEach-Object {
$ExperimentalFeaturesFromGalleryModulesInPSHome = @()
Copy link
Author

Choose a reason for hiding this comment

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

The PSDesiredStateConfiguration.InvokeDscResource exp.feature is no longer there - that is intentional, but I don't want to get rid of this code because it might be useful in the future to other modules, so leaving just empty array here.

@anmenaga anmenaga added the Breaking-Change breaking change that may affect users label Jun 7, 2021
@rjmholt rjmholt requested a review from bagajjal June 7, 2021 22:05
@iSazonov iSazonov added the Documentation Needed in this repo Documentation is needed in this repo label Jun 8, 2021
@iSazonov
Copy link
Collaborator

iSazonov commented Jun 8, 2021

I added Documentation-Needed label - I think we need to clearly document this in release documentation. If you agree please open new issue in PowerShell-Docs repository. I don't think a blog post is enough.

@anmenaga
Copy link
Author

anmenaga commented Jun 8, 2021

@anmenaga anmenaga changed the title WIP: Remove PSDesiredStateConfiguration v2.0.5 module from PS releases. Remove PSDesiredStateConfiguration v2.0.5 module from PS releases. Jun 8, 2021
@anmenaga anmenaga added this to the 7.2.0-preview.7 milestone Jun 9, 2021
@ghost ghost removed this from the 7.2.0-preview.7 milestone Jun 9, 2021
@ghost
Copy link

ghost commented Jun 9, 2021

Open PRs should not be assigned to milestone, so they are not assigned to the wrong milestone after they are merged. For backport consideration, use a backport label.

@TravisEz13 TravisEz13 assigned TravisEz13 and unassigned rjmholt Jun 9, 2021
@TravisEz13 TravisEz13 changed the title Remove PSDesiredStateConfiguration v2.0.5 module from PS releases. Remove PSDesiredStateConfiguration v2.0.5 module Jun 9, 2021
@TravisEz13 TravisEz13 merged commit 6e73c2d into PowerShell:master Jun 9, 2021
@TravisEz13 TravisEz13 added this to the 7.2.0-preview.7 milestone Jun 9, 2021
@TravisEz13 TravisEz13 added the CL-BreakingChange Indicates that a PR should be marked as a breaking change in the Change Log label Jun 9, 2021
@ghost
Copy link

ghost commented Jun 17, 2021

🎉v7.2.0-preview.7 has been released which incorporates this pull request.:tada:

Handy links:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Breaking-Change breaking change that may affect users CL-BreakingChange Indicates that a PR should be marked as a breaking change in the Change Log CL-BuildPackaging Indicates that a PR should be marked as a build or packaging change in the Change Log Documentation Needed in this repo Documentation is needed in this repo

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants