Skip to content

Renamed FoldingRangeRequestParam => FoldingRangeParams#550

Merged
dbaeumer merged 1 commit intomicrosoft:gh-pagesfrom
TypeFox:msp_foldingRangeParam
Aug 29, 2018
Merged

Renamed FoldingRangeRequestParam => FoldingRangeParams#550
dbaeumer merged 1 commit intomicrosoft:gh-pagesfrom
TypeFox:msp_foldingRangeParam

Conversation

@spoenemann
Copy link
Contributor

For all other requests, the single parameter type is named ...Params. For consistency, this pattern should also apply to the new folding range request.

@spoenemann
Copy link
Contributor Author

By the way, the export keyword doesn't make any sense in the specification. Should it be stripped off?

@rcjsuen
Copy link
Contributor

rcjsuen commented Aug 23, 2018

Personally, I would prefer changing it to FoldingRangeParams and dropping Request from the name. Outside of ShowMessageRequestParams, all the other parameters do not include the word Request in its interface name.

@dbaeumer
Copy link
Member

+1 for removing request

@spoenemann spoenemann force-pushed the msp_foldingRangeParam branch from f082821 to 533a4b9 Compare August 23, 2018 14:11
@spoenemann spoenemann changed the title Renamed FoldingRangeRequestParam => FoldingRangeRequestParams Renamed FoldingRangeRequestParam => FoldingRangeParams Aug 23, 2018
@spoenemann
Copy link
Contributor Author

Done.

@dbaeumer
Copy link
Member

@aeschli can you please decide on this. We should do this sooner than later

@dbaeumer dbaeumer merged commit 24b229a into microsoft:gh-pages Aug 29, 2018
aeschli added a commit to microsoft/vscode-languageserver-node that referenced this pull request Aug 29, 2018
@spoenemann spoenemann deleted the msp_foldingRangeParam branch August 29, 2018 11:19
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.

4 participants