[THRIFT-5032] Allows PascalCase properties for netstd

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 0.14.0
• Component/s: netstd - Compiler
• Assignee: Paulo Neves
• Reporter: Paulo Neves
• Created: 25/Nov/19 17:59
• Updated: 11/Feb/21 22:26
• Resolved: 28/Nov/19 20:38

Description

Microsoft guidelines  state that public properties should be PascalCase. Currently if a property in the thrift description is MyProperty it will be expanded into My_property which is clearly wrong. As this is a breaking change I added a pascal option to the netstd generator. I believe that in the future this should be the default and the old style deprecated.

I also added the option to the generator of the Server test, so that we can have a test with the properties in PascalCase talking with a netstd client without PascalCase. I hope this is ok.

 

https://docs.microsoft.com/en-us/dotnet/standard/design-guidelines/capitalization-conventions

Issue Links

• links to: GitHub Pull Request #1949