Skip to content

fix: check if git exists without calling the program for windows install - #178

Merged
zimeg merged 13 commits into
mainfrom
zimeg-fix-install-windows
Feb 26, 2026
Merged

fix: check if git exists without calling the program for windows install#178
zimeg merged 13 commits into
mainfrom
zimeg-fix-install-windows

Conversation

@zimeg

@zimegzimeg commented Aug 7, 2025

Copy link
Copy Markdown
Member

Summary

This PR checks for if an installed git command exists on Windows. Fixes an issue where calling the installed git command without arguments caused the program to return with an error after completing the installation!

Reviewers

The following commands can be used to test these changes! 🔍

irm https://downloads.slack-edge.com/slack-cli/install-windows-dev.ps1 -outfile 'install-windows-dev.ps1'
.\install-windows-dev.ps1 -Version 3.5.2

Requirements

@zimeg
zimeg requested a review from ewanek1August 7, 2025 06:20
@zimegzimeg self-assigned this Aug 7, 2025
@zimegzimeg added bug M-T: confirmed bug report. Issues are confirmed when the reproduction steps are documented semver:patch Use on pull requests to describe the release version increment build M-T: Changes to compilation and CI processes labels Aug 7, 2025
@codecov

codecovBot commented Aug 7, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 64.53%. Comparing base (cf7214c) to head (0c3d780).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files
@@ Coverage Diff @@## main #178 +/- ##
==========================================
- Coverage 64.57% 64.53% -0.04% 
==========================================
Files 213 213 Lines 17915 17915 ==========================================
- Hits 11568 11561 -7 - Misses 5273 5278 +5 - Partials 1074 1076 +2 

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@zimegzimeg left a comment

Copy link
Copy Markdown
MemberAuthor

Choose a reason for hiding this comment

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

🗣️ Update on happenings of this PR.

Comment threadscripts/install-windows-dev.ps1 Outdated
delay 0.3 "Now checking if it's the same Slack CLI..."
}
& $SLACK_CLI_NAME _fingerprint | Tee-Object -Variable get_finger_print | Out-Null
& $SLACK_CLI_NAME --version | Tee-Object -Variable slack_cli_version | Out-Null

Copy link
Copy Markdown
MemberAuthor

Choose a reason for hiding this comment

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

👁️‍🗨️ note: Great findings from @ewanek1 hint that this _fingerprint might be causing CI to hang on a first installation?

@zimegzimeg changed the title fix: hide version checks at the end of windows install scriptfix: check if git exists without calling the program for windows installFeb 25, 2026
@zimegzimeg added this to the Next Release milestone Feb 26, 2026
@zimeg
zimeg marked this pull request as ready for review February 26, 2026 21:23
@zimeg
zimeg requested a review from a team as a code ownerFebruary 26, 2026 21:23
@zimeg
zimeg requested review from hello-ashleyintech and removed request for ewanek1February 26, 2026 21:34

@hello-ashleyintechhello-ashleyintech left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

windows getting some luv 🚢

@zimeg

Copy link
Copy Markdown
MemberAuthor

@hello-ashleyintech No doubt! It feels nice to be finding cause of this after so much search 🔍 ✨

Huge thanks for the review and efforts of @ewanek1 for getting us started on this before reports were raised. Let's get this merged!

@zimeg
zimeg merged commit 54ba6d6 into mainFeb 26, 2026
7 checks passed
@zimeg
zimeg deleted the zimeg-fix-install-windows branch February 26, 2026 21:57
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugM-T: confirmed bug report. Issues are confirmed when the reproduction steps are documentedbuildM-T: Changes to compilation and CI processessemver:patchUse on pull requests to describe the release version increment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@zimeg@hello-ashleyintech