Skip to content
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

Jump to bottom

C# codegen should put packages into PascalCase for namespace defaulting #312

Closed
jskeet opened this issue Apr 28, 2015 · 1 comment
Closed

C# codegen should put packages into PascalCase for namespace defaulting #312

jskeet opened this issue Apr 28, 2015 · 1 comment
Labels
c#
Milestone
C# code cleanup

Comments

@jskeet
Copy link
Contributor

jskeet commented Apr 28, 2015

Currently, unless the csharp_namespace option is used, the package name is used directly for the namespace. It would be better if this were PascalCased. So for example:

google.protobuf => Google.Protobuf
noda_time.proto => NodaTime.Proto
@anandolee anandolee added the c# label Apr 28, 2015
@jskeet jskeet added c# and removed c# labels Apr 29, 2015
@jtattermusch jtattermusch added this to the C# code cleanup milestone Jun 19, 2015
@jtattermusch
Copy link
Contributor

It should be fine to do this now, as this won't break any users.

@jskeet jskeet mentioned this issue Jul 10, 2015
Merged
@seniorquico seniorquico mentioned this issue Aug 28, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
c#
Projects
None yet
Milestone
C# code cleanup
Development

No branches or pull requests
3 participants
@jskeet @jtattermusch @anandolee