Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

update enums article#116

Merged
ammbra merged 4 commits into
java:mainjava/devjava-content:mainfrom
danthe1st:enum-updatedanthe1st/devjava-content:enum-updateCopy head branch name to clipboard
Sep 25, 2024
Merged

update enums article#116
ammbra merged 4 commits into
java:mainjava/devjava-content:mainfrom
danthe1st:enum-updatedanthe1st/devjava-content:enum-updateCopy head branch name to clipboard

Conversation

@danthe1st

Copy link
Copy Markdown
Contributor

This PR contains minor updates to the enums article I originally contributed in #45.

I made changes to the writing (e.g. "exhaustiveness checking" instead of "exhaustiveness") and
I made the following changes:

  • use the plural of enum constants in a comment about listing them
  • use the name "exhaustiveness checking" (should that be Title Case?) instead of just the concept of exhaustiveness as well as add a link to the patternmatching article (The article isn't available in this repository so I didn't use an id: link)
  • move the enum constants of the enum with more than the enum declaration in the class body to two lines. This makes it easier to see the ; after the enum declaration.
  • add a few spaces in code as most people prefer that I think
  • slightly reword the conclusion

@oracle-contributor-agreement oracle-contributor-agreement Bot added the OCA Verified All contributors have signed the Oracle Contributor Agreement. label Jun 28, 2024

@ammbra ammbra left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Hey Dan 👋 ,

Thank you for contributing back with these changes. Can you please take a look at my suggestions (there is no need to add capital letters in the beginning of switch expressions, sealed classes etc).

@danthe1st

Copy link
Copy Markdown
Contributor Author

Thank you for your review. I addressed the feedback.

@danthe1st

Copy link
Copy Markdown
Contributor Author

Is there anything else I should change?

@ammbra

ammbra commented Jul 15, 2024

Copy link
Copy Markdown
Collaborator

All good.

@ammbra ammbra left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Looks good to me.

@ammbra ammbra merged commit f0fadfe into java:main Sep 25, 2024
@danthe1st danthe1st deleted the enum-update branch September 26, 2024 08:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

OCA Verified All contributors have signed the Oracle Contributor Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.