matchai / spacefish

🚀🐟 The fish shell prompt for astronauts
https://spacefish.matchai.dev
MIT License
964 stars 79 forks source link

feat: add `SPACEFISH_PROMPT_BOLD` option #153

Closed kiurchv closed 5 years ago

kiurchv commented 5 years ago

Description

This change adds an ability to toggle font weight of prompt via SPACEFISH_PROMPT_BOLD option

Motivation and Context

It will be great to have the ability to disable default bold styling of the prompt

Types of changes

Screenshots (if appropriate):

image

How Has This Been Tested?

Checklist:

matchai commented 5 years ago

Thank you @kiurchv for the contribution! 😄 I'd love to get this merged in as soon as the feature is approved in https://github.com/denysdovhan/spaceship-prompt/pull/519.

For the time being, would you be able to add tests for the new configuration option to __sf_lib_section.test.fish and __sf_section_dir.test.fish?

kiurchv commented 5 years ago

Hey, @matchai! I've added some tests and rebased my commits on the current master

matchai commented 5 years ago

Thank you @kiurchv! It appears there's still some uncertainty about how they plan to implement this feature upstream in spaceship, so I'm still a little hesitant to merge this in. 🤔

I'll keep an eye on the conversation and will keep you updated.

matchai commented 5 years ago

Closing this PR until the associated feature is implemented upstream.