Skip to content

developer-notes: add reference to snake_case and PascalCase #10917

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.

Sign up for GitHub

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
Merged
merged 1 commit into from
Jul 26, 2017
Merged

developer-notes: add reference to snake_case and PascalCase #10917

merged 1 commit into from
Jul 26, 2017

Conversation

benma
Copy link

@benma benma commented Jul 24, 2017

CamelCase is ambiguous as it refers to both lowerCamelCase and
UpperCamelCase, whereas PascalCase is only UpperCamelCase.

@benma
Copy link
Author

benma commented Jul 24, 2017

I realize that the way CamelCase was written, it implied UpperCamelCase, but I still did a double take as I have only ever seen camelCase refer to lowerCamelCase.

@fanquake fanquake added the Docs label Jul 24, 2017
@bytting
Copy link
Contributor

bytting commented Jul 24, 2017

Variable and namespace names are all lowercase, and may use _ to separate words.

This describes what is called "snake_case". Maybe the documentation should mention that too.

@benma
Copy link
Author

benma commented Jul 24, 2017

@corebob good idea; added.

@sipa
Copy link
Member

sipa commented Jul 24, 2017

Interesting; I had never heard of PascalCase, and only seen CamelCase to refer to it.

@benma
Copy link
Author

benma commented Jul 24, 2017

That is interesting ;) I made another revision: UpperCamelCase (PascalCase).

@sipa
Copy link
Member

sipa commented Jul 25, 2017

utACK

promag
promag reviewed Jul 25, 2017
View reviewed changes
Copy link
Contributor

@promag promag left a comment

Choose a reason for hiding this comment

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

Please rename PR title and commit to something like Add reference to snake_case and PascalCase in developer notes. Missing snake_case in PR description. Otherwise ACK.

developer-notes: add reference to snake_case and PascalCase
4a7a4ff 
CamelCase is ambiguous as it refers to both lowerCamelCase and
UpperCamelCase, whereas PascalCase is only UpperCamelCase.
@benma
Copy link
Author

benma commented Jul 25, 2017

@promag done

@benma benma changed the title developer-notes: CamelCase -> PascalCase developer-notes: add reference to snake_case and PascalCase Jul 25, 2017
@laanwj
Copy link
Member

laanwj commented Jul 26, 2017

PascalCase ❤️
utACK 4a7a4ff

@laanwj laanwj merged commit 4a7a4ff into bitcoin:master Jul 26, 2017
laanwj added a commit that referenced this pull request Jul 26, 2017
@laanwj
Merge #10917: developer-notes: add reference to snake_case and Pascal…
5c003cb 
…Case

4a7a4ff developer-notes: add reference to snake_case and PascalCase (Marko Bencun)

Pull request description:

  CamelCase is ambiguous as it refers to both lowerCamelCase and
  UpperCamelCase, whereas PascalCase is only UpperCamelCase.

Tree-SHA512: 8eebc42931f10ed8fd314c6b8a2a936aa18fce358a50bb8ae580404fb06a97b8fece12c0398170a9a8ddce250d1e79ece3774cf8a36ac604d67b2797b54aa005
@ryanofsky ryanofsky mentioned this pull request Nov 1, 2018
Closed
@benma benma deleted the pascalcase branch September 19, 2019 12:56
PastaPastaPasta pushed a commit to PastaPastaPasta/dash that referenced this pull request Sep 19, 2019
@laanwj @PastaPastaPasta
Merge bitcoin#10917: developer-notes: add reference to snake_case and…
ddd4b8b 
… PascalCase

4a7a4ff developer-notes: add reference to snake_case and PascalCase (Marko Bencun)

Pull request description:

  CamelCase is ambiguous as it refers to both lowerCamelCase and
  UpperCamelCase, whereas PascalCase is only UpperCamelCase.

Tree-SHA512: 8eebc42931f10ed8fd314c6b8a2a936aa18fce358a50bb8ae580404fb06a97b8fece12c0398170a9a8ddce250d1e79ece3774cf8a36ac604d67b2797b54aa005
PastaPastaPasta pushed a commit to PastaPastaPasta/dash that referenced this pull request Sep 23, 2019
@laanwj @PastaPastaPasta
Merge bitcoin#10917: developer-notes: add reference to snake_case and…
8e9d322 
… PascalCase

4a7a4ff developer-notes: add reference to snake_case and PascalCase (Marko Bencun)

Pull request description:

  CamelCase is ambiguous as it refers to both lowerCamelCase and
  UpperCamelCase, whereas PascalCase is only UpperCamelCase.

Tree-SHA512: 8eebc42931f10ed8fd314c6b8a2a936aa18fce358a50bb8ae580404fb06a97b8fece12c0398170a9a8ddce250d1e79ece3774cf8a36ac604d67b2797b54aa005
PastaPastaPasta pushed a commit to PastaPastaPasta/dash that referenced this pull request Sep 24, 2019
@laanwj @PastaPastaPasta
Merge bitcoin#10917: developer-notes: add reference to snake_case and…
db912de 
… PascalCase

4a7a4ff developer-notes: add reference to snake_case and PascalCase (Marko Bencun)

Pull request description:

  CamelCase is ambiguous as it refers to both lowerCamelCase and
  UpperCamelCase, whereas PascalCase is only UpperCamelCase.

Tree-SHA512: 8eebc42931f10ed8fd314c6b8a2a936aa18fce358a50bb8ae580404fb06a97b8fece12c0398170a9a8ddce250d1e79ece3774cf8a36ac604d67b2797b54aa005
PastaPastaPasta pushed a commit to PastaPastaPasta/dash that referenced this pull request Nov 19, 2019
@laanwj @PastaPastaPasta
Merge bitcoin#10917: developer-notes: add reference to snake_case and…
e0f1196 
… PascalCase

4a7a4ff developer-notes: add reference to snake_case and PascalCase (Marko Bencun)

Pull request description:

  CamelCase is ambiguous as it refers to both lowerCamelCase and
  UpperCamelCase, whereas PascalCase is only UpperCamelCase.

Tree-SHA512: 8eebc42931f10ed8fd314c6b8a2a936aa18fce358a50bb8ae580404fb06a97b8fece12c0398170a9a8ddce250d1e79ece3774cf8a36ac604d67b2797b54aa005
PastaPastaPasta pushed a commit to PastaPastaPasta/dash that referenced this pull request Dec 9, 2019
@laanwj @PastaPastaPasta
Merge bitcoin#10917: developer-notes: add reference to snake_case and…
ce78509 
… PascalCase

4a7a4ff developer-notes: add reference to snake_case and PascalCase (Marko Bencun)

Pull request description:

  CamelCase is ambiguous as it refers to both lowerCamelCase and
  UpperCamelCase, whereas PascalCase is only UpperCamelCase.

Tree-SHA512: 8eebc42931f10ed8fd314c6b8a2a936aa18fce358a50bb8ae580404fb06a97b8fece12c0398170a9a8ddce250d1e79ece3774cf8a36ac604d67b2797b54aa005
PastaPastaPasta pushed a commit to PastaPastaPasta/dash that referenced this pull request Jan 1, 2020
@laanwj @PastaPastaPasta
Merge bitcoin#10917: developer-notes: add reference to snake_case and…
6a6c267 
… PascalCase

4a7a4ff developer-notes: add reference to snake_case and PascalCase (Marko Bencun)

Pull request description:

  CamelCase is ambiguous as it refers to both lowerCamelCase and
  UpperCamelCase, whereas PascalCase is only UpperCamelCase.

Tree-SHA512: 8eebc42931f10ed8fd314c6b8a2a936aa18fce358a50bb8ae580404fb06a97b8fece12c0398170a9a8ddce250d1e79ece3774cf8a36ac604d67b2797b54aa005
PastaPastaPasta pushed a commit to PastaPastaPasta/dash that referenced this pull request Jan 2, 2020
@laanwj @PastaPastaPasta
Merge bitcoin#10917: developer-notes: add reference to snake_case and…
cca4376 
… PascalCase

4a7a4ff developer-notes: add reference to snake_case and PascalCase (Marko Bencun)

Pull request description:

  CamelCase is ambiguous as it refers to both lowerCamelCase and
  UpperCamelCase, whereas PascalCase is only UpperCamelCase.

Tree-SHA512: 8eebc42931f10ed8fd314c6b8a2a936aa18fce358a50bb8ae580404fb06a97b8fece12c0398170a9a8ddce250d1e79ece3774cf8a36ac604d67b2797b54aa005
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Dec 16, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
1 more reviewer

@promag promag promag left review comments

Reviewers whose approvals may not affect merge requirements
Assignees
No one assigned
Labels
Docs
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@benma @bytting @sipa @laanwj @promag @fanquake