refactor: Allow starship to be better used programmatically#509
Merged
matchai merged 1 commit intostarship:masterfrom Oct 9, 2019
Southclaws:prompt-api
Merged
refactor: Allow starship to be better used programmatically#509matchai merged 1 commit intostarship:masterfrom Southclaws:prompt-api
matchai merged 1 commit intostarship:masterfrom
Southclaws:prompt-api
Conversation
…stead of writing directly to stdout. This makes it easier to embed Starship into other Rust programs such as shells written in Rust. It also decouples the arguments from the context for more programmatic initialisation of the context.
|
Speaking from the nushell side, we're definitely interested in making these work together so that nushell users can use starship as their prompt. |
matchai
approved these changes
Oct 9, 2019
Member
matchai
left a comment
There was a problem hiding this comment.
Looks good to me! 👍
Feel free to drop us a line if we could do anything to simplify integration with nushell in the future. 😄
Member
|
@all-contributors Please add @Southclaws for code. |
Contributor
|
I've put up a pull request to add @Southclaws! 🎉 |
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
3 participants
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.
Description
This makes it easier to embed Starship into other Rust programs such as shells written in Rust. It also decouples the arguments from the context for more programmatic initialisation of the context.
Motivation and Context
Partially this issue: nushell/nushell#356 and also just making it easier to use Starship as a Rust library for those building shells in Rust.
Types of changes
How Has This Been Tested?
Kind of N/A I suppose since the change is internal and tiny.
Checklist: