-
-
Notifications
You must be signed in to change notification settings - Fork 803
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
Refactor Settings & Write-GitStatus function #513
Merged
Merged
Changes from 1 commit
Commits
Show all changes
24 commits
Select commit
Hold shift + click to select a range
a7b01c5
WIP: A simpler take at organizing settings using v5 classes
rkeithhill 7460daf
Reorder some settings
rkeithhill 98c61bb
Make sure value is not null
rkeithhill b28513c
Try another approach with custom ansi
rkeithhill 4d8e08c
WIP - refactor Write-Prompt
rkeithhill 8fcfc21
WIP refactor Write-GitStatus into individual Write-* methods.
rkeithhill 7fb0d6d
Prefix new function nouns with "Git"
rkeithhill 8dee3e7
Rename GitWorkingDirectory to GitWorkingDir
rkeithhill 71a085a
Favor ConsoleColor over HtmlColor
rkeithhill fa2b05b
Change var name from $res to $prompt
rkeithhill facc87d
Convert from untyped settings to strongly typed settings
rkeithhill 5120624
Add Write-GitBranchName
rkeithhill 88312b2
Fix a typo in help comment
rkeithhill b656d1e
Prep branch for a possible publish as Beta1
rkeithhill db734a6
Export $GitPromptScriptBlock variable
rkeithhill b7b58bc
Merge from develop
rkeithhill 53cb658
Fix Write-VcsStatus issue with returning array of prompt strings
rkeithhill a3a4ffe
Address PR feedback on -Status param help
rkeithhill 2af60e0
Address PR feedback to move types to separate file
rkeithhill 6235734
Adjust Prerelease field - set to `alpha`
rkeithhill 9f9a649
Address PR feedback - simplify Write-VcsStatus
rkeithhill 635c113
Address PR feedback - DarkCyan back to Cyan
rkeithhill 041c286
Fix bug where color 'Black' wasn't matching ConsoleColor
rkeithhill 2831c0c
Add -Color parameter to Write-Prompt
rkeithhill 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
There are no files selected for viewing
This file contains 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
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I may be oversimplifying, but isn't this the same as:
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Probably not. I was following the pattern we'd established in Write-Git*Status methods. Just thinking of the case where AnsiConsole is disabled - that should write to the host and not return anything (or empty strings) and those should concat to an empty string. So yeah, that should work. Want me to change this (and test it)?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure, I'm all for less code. 99.99+% of the time there's a single status function with a single result.