Skip to content

[A] fix installer to validate correctly the checksum#925

Merged
hugomontero merged 1 commit into
masterfrom
bug/cli-installer-unix-checksum
Jun 16, 2026
Merged

[A] fix installer to validate correctly the checksum#925
hugomontero merged 1 commit into
masterfrom
bug/cli-installer-unix-checksum

Conversation

@hugomontero

@hugomontero hugomontero commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

Description

Fix unix installer for particle-cli sha-256 check.

  • Eyeballing the installer script at installer/unix/install-cli shows that you're using 'shasum a 256... ' instead of 'shasum -a 256... ' on line:
  • Also the sha was checking the uncompressed file instead of the compressed one/

How to Test

  • Try to use the sh file to install the cli.

outcome

  • Sha verification should happen
  • it will validate the compressed file instead of the uncompressed one.

Related Issues / Discussions

Issue: #909

Completeness

  • User is totes amazing for contributing!
  • Contributor has signed CLA
  • Problem and solution clearly stated
  • Tests have been provided
  • Docs have been updated
  • CI is passing

@hugomontero hugomontero force-pushed the bug/cli-installer-unix-checksum branch from 94ee9e6 to d85018f Compare June 15, 2026 16:09
@hugomontero hugomontero marked this pull request as ready for review June 15, 2026 21:08
@hugomontero hugomontero requested a review from monkbroc June 15, 2026 21:08

@monkbroc monkbroc left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Tested locally 👍

Base automatically changed from feature/sc-140095/verify-every-cli-command-and-make-sure-they to master June 16, 2026 14:42
@hugomontero hugomontero force-pushed the bug/cli-installer-unix-checksum branch from d85018f to 8157ac3 Compare June 16, 2026 14:43
@hugomontero hugomontero merged commit f60b243 into master Jun 16, 2026
5 checks passed
@hugomontero hugomontero deleted the bug/cli-installer-unix-checksum branch June 16, 2026 15:00
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