Add doNotPerformUpgrade setting and enhance upgrade warning messages#2327
Open
ChrisBlankDe wants to merge 12 commits into
Open
Add doNotPerformUpgrade setting and enhance upgrade warning messages#2327ChrisBlankDe wants to merge 12 commits into
ChrisBlankDe wants to merge 12 commits into
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
Adds doNotPerformUpgrade to skip upgrade execution while retaining publishing and testing behavior.
Changes:
- Adds and forwards the new pipeline setting.
- Updates upgrade-related warnings and documentation.
- Adds schema validation coverage.
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
Actions/.Modules/ReadSettings.psm1 |
Initializes the setting to false. |
Actions/.Modules/settings.schema.json |
Defines the boolean setting. |
Actions/RunPipeline/RunPipeline.ps1 |
Forwards the setting and clarifies warnings. |
Tests/ReadSettings.Test.ps1 |
Verifies schema presence and type. |
Scenarios/settings.md |
Documents the setting. |
RELEASENOTES.md |
Announces the feature. |
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Contributor
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 6 out of 6 changed files in this pull request and generated no new comments.
Comments suppressed due to low confidence (1)
Tests/ReadSettings.Test.ps1:547
- Close the new
Itblock before the existing validation test. As written, the secondItis nested and the file ends with an unmatched{, so this Pester file cannot be parsed or run.
It 'ValidateSettings skips validation entirely on PS versions less than 7 without warning' {
mazhelez
approved these changes
Jul 22, 2026
mazhelez
self-requested a review
July 22, 2026 18:54
mazhelez
previously approved these changes
Jul 23, 2026
…ine in ReadSettings.Test.ps1 Copilot-Session: 2add9b5c-0d44-4b0e-a260-c25400a7bc05
aholstrup1
approved these changes
Jul 23, 2026
mazhelez
approved these changes
Jul 23, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
❔What, Why & How
This pull request introduces a new pipeline setting,
doNotPerformUpgrade, which allows users to skip the upgrade phase inRun-AlPipelinewhile continuing to publish and test apps. The setting is integrated into the configuration schema, documented for users, and tested for presence in the schema. Additionally, warning messages related to upgrade testing have been improved for clarity.New Feature: Skip Upgrade Phase
doNotPerformUpgradeboolean setting to the default settings inReadSettings.psm1, the pipeline schema (settings.schema.json), and pipeline parameter list inRunPipeline.ps1, allowing users to bypass the upgrade phase during pipeline execution.RELEASENOTES.mdandScenarios/settings.md, explaining its purpose and usage.doNotPerformUpgradeis present in the settings schema.Pipeline Usability Improvements
RunPipeline.ps1to clarify that missing previous release apps disables the AppSourceCop breaking change check.Original PR #2297 was reverted with PR #2305. This PR matches the original, with the additional initialization of the settings.
Related to issue: #2296
✅ Checklist