Skip to content
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

fix: autocomplete script comparison in interactie shell #493

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

Conversation

miki725
Copy link
Collaborator

@miki725 miki725 commented Feb 25, 2025

Issue

fixes #251

Description

The autocomplete script is now chalked on-disk during installation from chalk metadata vs embedding that metadata to script which is embedded in the repo. This required some rework of the source scripts plugin along with some collection logic such as to ignore all irrelevant plugins when copying metadata from one chalk object to another.

Testing

  • compile chalk
  • ./chalk version
  • rm -rf ~/.local/share/bash_completion/completions/chalk.bash; ./chalk version

check that its only updated when necessary

instead of hardcoding autocomplete version in chalk codebase, use chalk
version itself
@miki725 miki725 force-pushed the autocomplete branch 2 times, most recently from d005f1f to ef52e10 Compare February 26, 2025 13:27
autocomplete now fully uses that codec to update the autocomplete shell
script on system when using TTY session
instead of 2 versions of autocomplete, make the default one more generic
with if statements so its more reliable
@miki725 miki725 marked this pull request as ready for review February 27, 2025 17:48
@miki725 miki725 requested a review from viega as a code owner February 27, 2025 17:48
@miki725 miki725 requested a review from mbaltrusitis February 28, 2025 13:16
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.

Chalk reporting invalid chalk mark in autocomplete script
1 participant