Skip to content

Add support for rocket and eyes reactions. - #707

Merged
bitwiseman merged 2 commits into
hub4j:masterfrom
rdvdijk:add-rocket-eyes-reactions
Feb 23, 2020
Merged

Add support for rocket and eyes reactions.#707
bitwiseman merged 2 commits into
hub4j:masterfrom
rdvdijk:add-rocket-eyes-reactions

Conversation

@rdvdijk

@rdvdijk rdvdijk commented Feb 21, 2020

Copy link
Copy Markdown
Contributor

Description

Added the missing reactions 🚀 and 👀.

Before submitting a PR:

We love getting PRs, but we hate asking people for the same basic changes every time.

  • Push your changes to a branch other than master. Create your PR from that branch.
  • Add JavaDocs and other comments (already present)
  • Write tests that run and pass in CI. See CONTRIBUTING.md for details on how to capture snapshot data. (there are no tests for this enum)
  • Run mvn -D enable-ci clean install site locally. This may reformat your code, commit those changes. If this command doesn't succeed, your change will not pass CI. (master branch fails on coverage checks)

@rdvdijk
rdvdijk requested a review from bitwiseman February 21, 2020 12:22
@rdvdijk rdvdijk changed the title Add support for rocket and eyes reactions. Add support for 🚀 and eyes reactions. Feb 21, 2020
@rdvdijk rdvdijk changed the title Add support for 🚀 and eyes reactions. Add support for rocket and eyes reactions. Feb 21, 2020

@bitwiseman bitwiseman left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Cool! Please add a test.

@bitwiseman

Copy link
Copy Markdown
Member

(I'll fix master momentarily.)

@rdvdijk

rdvdijk commented Feb 21, 2020

Copy link
Copy Markdown
Contributor Author

Cool! Please add a test.

Will do! Will a unit test verifying the forContent() for all enum values method suffice?

@bitwiseman

Copy link
Copy Markdown
Member

@rdvdijk
Yes that would be great.

@bitwiseman

Copy link
Copy Markdown
Member

@rdvdijk I got it. Thanks for the submission!

@bitwiseman
bitwiseman merged commit 0c3c490 into hub4j:master Feb 23, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants