-
Notifications
You must be signed in to change notification settings - Fork 137
Shell completion auto-install and pre-commit hook improvements #1124
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
Merged
Merged
Changes from 18 commits
Commits
Show all changes
52 commits
Select commit
Hold shift + click to select a range
7496ab7
Add CI lint gate and local precheck command
sbryngelson 26f7e17
Fix precheck.sh portability and usability issues
sbryngelson 4b17fa2
Gate benchmarks on Test Suite completion
sbryngelson 1b211a5
Auto-install git pre-commit hook for precheck
sbryngelson 9304137
Use dynamic CPU count in pre-commit hook
sbryngelson 77ac8cc
Show CPU count in pre-commit hook output
sbryngelson a7cf1d2
Merge upstream master, resolve mfc.sh conflict
sbryngelson 4b55c30
Add precheck command to CLI and autocomplete
sbryngelson e017cf0
Auto-update installed shell completions on regeneration
sbryngelson 780138b
Show source command when completions auto-update
sbryngelson 06f0425
Always check installed completions on every run
sbryngelson f65ed30
Prevent directory completion fallback in shell completions
sbryngelson d23addc
Auto-install completions and fix bash completion options
sbryngelson 33cbd0e
Auto-install completions from mfc.sh with shell rc setup
sbryngelson 1ba4666
Clarify verbose, debug, and debug-log flag documentation
sbryngelson adfcb7f
Cap pre-commit hook parallelism at 12 jobs
sbryngelson 02ef3de
Fix completion auto-install to check for files, not just directory
sbryngelson ff8d499
Merge MFlowCode/MFC master into triage
sbryngelson fc5e2a5
Auto-update outdated shell completions
sbryngelson 9c15c8a
Auto-activate completions when mfc.sh is sourced
sbryngelson 52d0ddd
Use workflow_run for benchmarks and extract completion logic
sbryngelson 8e3f405
Suppress verbose package list from uv install by default
sbryngelson 989a79d
Compact splash screen from 45 to 20 lines
sbryngelson e913f7c
Pass arguments to python.sh for verbose flag support
sbryngelson 6972e6e
Handle ./mfc.sh -v without command (show help, not error)
sbryngelson cf49bcc
Fix splash screen to use COMMANDS as single source of truth
sbryngelson 72d8a38
Address AI reviewer feedback on PR #1124
sbryngelson 88d8736
Show full build error output instead of truncating to 40 lines
sbryngelson c96edf2
Fix troubleshooting tips to suggest --debug instead of --debug-log
sbryngelson b44360f
Fix flags-before-command detection and shell detection
sbryngelson f24c62b
Raise minimum Python version to 3.12 (pyrometheus requires it)
sbryngelson e0bdc88
Lower minimum Python to 3.10 by pinning pyrometheus to pre-3.12 commit
sbryngelson f9fa919
Add -v flag to all CI build/run/test/bench commands
sbryngelson 0ec406b
Add -v flag to coverage and GitHub runner CI commands
sbryngelson 1e863ae
Revert pyrometheus pin to track git HEAD
sbryngelson 73e0de3
Remove -v flag from HPC CI scripts to fix build stalls
sbryngelson b76f0a3
Fix shell detection for zsh users sourcing under bash
sbryngelson 13fcc83
Limit push-triggered CI to master branch only
sbryngelson 6b0faef
Fix shell detection to not use $SHELL fallback when in bash
sbryngelson 2651c08
Change Frontier SLURM account from CFD154 to ENG160
sbryngelson 18548ab
Remove -v flag from GitHub/coverage CI and pin pyrometheus
sbryngelson 32c4fa4
Harden CI against transient Frontier/Phoenix failures
sbryngelson cd1b771
Add SIGHUP protection to Frontier build scripts
sbryngelson 4353e1e
Restore -v flag to all CI build and test commands
sbryngelson 40a12e1
Remove -v from GitHub-hosted CI, keep for self-hosted runners
sbryngelson 0fdd27e
Fix pipe deadlock in -v build mode and restore -v everywhere
sbryngelson 8321eac
Pass --test-all to Build step so post_process builds during dry-run
sbryngelson f213175
Refactor test.yml: generic self-hosted steps via matrix.cluster
sbryngelson 3470c7a
Unpin pyrometheus to track git HEAD
sbryngelson d169d08
Clean build directory between retry attempts
sbryngelson 9734b2d
Add build retry logic with clean to Phoenix test script
sbryngelson 3fefb70
Add build retry with clean to Phoenix benchmark script
sbryngelson File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Some comments aren't visible on the classic Files Changed page.
There are no files selected for viewing
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
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.