Skip to content

Conversation

@rdvdijk
Copy link
Contributor

@rdvdijk rdvdijk commented Feb 21, 2020

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
Copy link
Member

@bitwiseman bitwiseman left a comment

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
Member

(I'll fix master momentarily.)

@rdvdijk
Copy link
Contributor Author

rdvdijk commented Feb 21, 2020

Cool! Please add a test.

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

@bitwiseman
Copy link
Member

@rdvdijk
Yes that would be great.

@bitwiseman
Copy link
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