Skip to content

Rename members to GetValue - #1787

Merged
jonsequitur merged 1 commit into
dotnet:mainfrom
Keboo:fix1785
Oct 31, 2022
Merged

Rename members to GetValue#1787
jonsequitur merged 1 commit into
dotnet:mainfrom
Keboo:fix1785

Conversation

@Keboo

@Keboo Keboo commented Jul 1, 2022

Copy link
Copy Markdown
Member

Added GetValue methods to InvocationContext
Renamed GetValueForOption and GetValueForArgument on ParseResult and SymbolResult classes

Fixes #1785

@jonsequitur jonsequitur left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Updated for breaking change in PR

@Keboo
Keboo force-pushed the fix1785 branch 3 times, most recently from 152c397 to a665e15 Compare July 12, 2022 19:33
Added GetValue methods to InvocationContext
Renamed GetValueForOption and GetValueForArgument on ParseResult and SymbolResult classes
Fixing compilation tests
@jonsequitur
jonsequitur merged commit b8ddb00 into dotnet:main Oct 31, 2022
@jonsequitur

Copy link
Copy Markdown
Contributor

Thanks, @Keboo!

@Keboo
Keboo deleted the fix1785 branch October 31, 2022 17:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Rename GetValueForOption and GetValueForArgument to GetValue and add these methods to InvocationContext

2 participants