Skip to content

Check if apps are already installed on a higher version before deploying #1756

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

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

mazhelez
Copy link
Collaborator

@mazhelez mazhelez commented Jun 2, 2025

Emit warnings when apps are installed on a higher version when deploying.

Issue #1722

@mazhelez mazhelez marked this pull request as ready for review June 11, 2025 07:42
@Copilot Copilot AI review requested due to automatic review settings June 11, 2025 07:42
@mazhelez mazhelez requested a review from a team as a code owner June 11, 2025 07:42
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds functionality to warn when an app is deployed with a version lower than an already installed higher version, addressing Issue #1722.

  • Updates the release notes with Issue #1722 details.
  • Introduces the CheckInstalledApps function in Deploy.ps1 to handle version comparisons between installed apps and app files.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
RELEASENOTES.md Adds a new issue entry for Issue #1722.
Actions/Deploy/Deploy.ps1 Adds a function to check installed app versions and emit warnings if necessary.
Comments suppressed due to low confidence (1)

Actions/Deploy/Deploy.ps1:67

  • The comment does not match the logic implemented; update it to indicate that a warning is emitted when the installed version is higher than the version in the app file.
# Check if the version of the installed app is lower than the version in the app file

aholstrup1
aholstrup1 previously approved these changes Jun 11, 2025
aholstrup1
aholstrup1 previously approved these changes Jun 17, 2025
@mazhelez mazhelez enabled auto-merge (squash) June 17, 2025 09:49
@mazhelez mazhelez requested a review from spetersenms July 2, 2025 10:50
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.

2 participants