Skip to content

Update PublishCodeCoverage to v2#552

Merged
gaelcolas merged 11 commits intogaelcolas:mainfrom
dan-hughes:fix-issue-551
Mar 30, 2026
Merged

Update PublishCodeCoverage to v2#552
gaelcolas merged 11 commits intogaelcolas:mainfrom
dan-hughes:fix-issue-551

Conversation

@dan-hughes
Copy link
Copy Markdown
Contributor

@dan-hughes dan-hughes commented Feb 11, 2026

Pull Request

Pull Request (PR) description

Update PublishCodeCoverage in azure-pipelines to v2. This is already validated in WSManDsc.

Task list

  • The PR represents a single logical change. i.e. Cosmetic updates should go in different PRs.
  • Added an entry under the Unreleased section of in the CHANGELOG.md as per format.
  • Local clean build passes without issue or fail tests (build.ps1 -ResolveDependency).
  • Documentation added/updated in README.md.
  • Comment-based help added/updated.
  • Localization strings added/updated in all localization files as appropriate.
  • Unit tests added/updated. See DSC Community Testing Guidelines.
  • Integration tests added/updated (where possible). See DSC Community Testing Guidelines.
  • New/changed code adheres to DSC Community Style Guidelines.

This change is Reviewable

@dan-hughes
Copy link
Copy Markdown
Contributor Author

@johlju @gaelcolas, can someone trigger CI please.

@johlju
Copy link
Copy Markdown
Collaborator

johlju commented Feb 11, 2026

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@johlju
Copy link
Copy Markdown
Collaborator

johlju commented Feb 11, 2026

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@johlju
Copy link
Copy Markdown
Collaborator

johlju commented Feb 11, 2026

Maybe third time is the charm, seems there is an issue with the gallery.

@johlju
Copy link
Copy Markdown
Collaborator

johlju commented Feb 11, 2026

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@johlju
Copy link
Copy Markdown
Collaborator

johlju commented Feb 11, 2026

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@johlju
Copy link
Copy Markdown
Collaborator

johlju commented Feb 11, 2026

@gaelcolas there are intermittent error (connection issues) from PS Gallery, shall we skip those tests for PowerShellGet for now as the goal is to move to PSResourceGet (you mentioned you worked on it)?

@johlju
Copy link
Copy Markdown
Collaborator

johlju commented Feb 12, 2026

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@johlju
Copy link
Copy Markdown
Collaborator

johlju commented Feb 12, 2026

Now this fails with the same problem as in Sampler.GitHubTasks, so there is an issue with upstream somehow.

@johlju
Copy link
Copy Markdown
Collaborator

johlju commented Feb 19, 2026

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@dan-hughes
Copy link
Copy Markdown
Contributor Author

@johlju, again please?

@johlju
Copy link
Copy Markdown
Collaborator

johlju commented Feb 19, 2026

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@dan-hughes
Copy link
Copy Markdown
Contributor Author

@johlju, @gaelcolas, pick through this,

It works, but maybe not as intended.

PSResourceGet 1.0.1 looks to be completely unusable/broken, there must have been a backend PSGallery change that this does not like.

Using PSResourceGet latest 1.1.1, seems to have a bug which rears it's head when looking to package Plaster, which on version 1.1.4 contains two psd1 files in the root of the module both starting with the ModuleName.

Additionally PSResourceGet 1.1.1 seems to pull in a Plaster pre-release version labelled 2.0.0-alpha1 due to Sampler declaring Plaster as a required module in it's manifest. It could be due to the same bug of not being able to determine the valid module manifest.

Seemingly, using PSResourceGet 1.2.0-rc3 fixes all of these issues...

@johlju
Copy link
Copy Markdown
Collaborator

johlju commented Feb 19, 2026

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@johlju
Copy link
Copy Markdown
Collaborator

johlju commented Feb 19, 2026

Great work @dan-hughes

So if 1.2.0-rc3 solves this, we should wait for 1.2.0 to be released so we don't have to pin an rc version?

@dan-hughes
Copy link
Copy Markdown
Contributor Author

dan-hughes commented Feb 19, 2026

Or don't do a full release? I wonder if Sampler.GitHubTasks was the same issue.

I'll let you both decide what to do.

@dan-hughes
Copy link
Copy Markdown
Contributor Author

@johlju, PSResourceGet released 1.2.0. PR updated.

@gaelcolas
Copy link
Copy Markdown
Owner

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Copy Markdown
Owner

@gaelcolas gaelcolas left a comment

Choose a reason for hiding this comment

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

that looks good to me.
Lets see if it passes the tests before merging.

@gaelcolas gaelcolas merged commit a4282b1 into gaelcolas:main Mar 30, 2026
11 checks passed
@dan-hughes dan-hughes deleted the fix-issue-551 branch March 30, 2026 10:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Publish code coverage' version 1 (PublishCodeCoverageResults@1) is deprecated

3 participants