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

Added a configuration option to suffix/prefix the prompt #110

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

hallzy
Copy link
Contributor

@hallzy hallzy commented Feb 28, 2017

Now, you can add a prefix and/or suffix to your git-radar prompt that
will only show if you are in a git repo.

Note: The PROMPT_FORMAT line was changed to remove the leading space in
the string. With it in, if you define a prefix a space will still be
there even if the user may not want it. So it can be overridden by the
user with one of the new variables, but by default, the prefix still
adds that space anyways.

Copy link
Owner

@michaeldfallen michaeldfallen left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution, sorry I've been less than responsive. Busy times at work.

README.md Outdated Show resolved Hide resolved
radar-base.sh Outdated Show resolved Hide resolved
Now, you can add a prefix and/or suffix to your git-radar prompt that
will only show if you are in a git repo.

Note: The PROMPT_FORMAT line was changed to remove the leading space in
the string. With it in, if you define a prefix a space will still be
there even if the user may not want it. So it can be overridden by the
user with one of the new variables, but by default, the prefix still
adds that space anyways.
@hallzy hallzy force-pushed the config_gitradar_trailing_char branch from e796c96 to 9c63abf Compare April 2, 2017 17:37
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