Skip to content
This repository has been archived by the owner on Oct 16, 2018. It is now read-only.

Enum constant names #39

Open
yole opened this issue Nov 17, 2017 · 1 comment
Open

Enum constant names #39

yole opened this issue Nov 17, 2017 · 1 comment

Comments

@yole
Copy link

yole commented Nov 17, 2017

The style guide doesn't talk about naming enum constants. They don't fall under "Constant values can only be defined inside of an object or as a top-level declaration", so they are not covered by that rule, and there should be a separate one for them.

(And please don't require UPPER_CASE for all enum constants; that would be ugly. Our style guide allows both UPPER_CASE and CamelCase.)

@hho
Copy link

hho commented Dec 18, 2017

@yole Please consider updating Kotlin's enum reference page to reflect that. Currently it looks like UPPER_CASE is the way to go.

(which I liked, because it's good for Java interop)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants