Merged
Conversation
Addresses tw93's PR tw93#305 feedback: - Add 'mo' short alias (mo.cmd) alongside mole.cmd - Use 'mo' in all help text and documentation - Document lowercase flag style (--dry-run, --help, etc.) - Simplify optimize: repairs run automatically, no extra flags - Fix RepairsApplied counter bug in optimize.ps1 - Update README with standardized examples
Fixes tw93#316 - The go.sum file was missing from the install file list, causing 'missing go.sum entry' errors when trying to build the Go tools (analyze, status) on systems without pre-built binaries.
There was a problem hiding this comment.
Pull request overview
This PR fixes issue #316 where mo analyze and mo status commands failed with "missing go.sum entry" errors when pre-built executables weren't present. The fix adds go.sum to the list of files copied by install.ps1, allowing Go to properly resolve dependencies when building the tools from source.
Changes:
- Fixed the installation script to copy
go.sumalongsidego.mod - Updated command-line interface to use
--flagformat instead of-Flagfor consistency - Added short alias support (
-h,-v, etc.) for common flags - Created
moas a short alias for themolecommand
Reviewed changes
Copilot reviewed 3 out of 9 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| install.ps1 | Added go.sum to the files copied during installation (fixes #316) and created mo.cmd alias |
| mole.ps1 | Updated flag parsing to support both single and double-dash formats, added short flag aliases |
| bin/analyze.ps1 | Updated help text to use mo command name and --help flag format |
| bin/clean.ps1 | Updated help text and added aliases for flags |
| bin/optimize.ps1 | Simplified by removing separate repair flags and integrating repairs into main optimization flow |
| bin/purge.ps1 | Updated help text and added flag aliases |
| bin/status.ps1 | Updated help text to use new command format |
| bin/uninstall.ps1 | Updated help text and added flag aliases |
| README.md | Updated all documentation examples to use mo command and new flag formats |
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.
Summary - Issue #316 Fix
Problem: mo analyze and mo status commands failed with "missing go.sum entry" error when the pre-built exe files weren't present.
Root Cause: The install.ps1 script was copying go.mod but not go.sum, so when Go tried to build the tools, it couldn't resolve dependencies.
Fix: Added go.sum to the list in install.ps1.