Skip to content
This repository has been archived by the owner on Mar 18, 2024. It is now read-only.

Added a few new rebuses #211

Merged

Conversation

William-Bowden
Copy link
Contributor

Associated Issue: #121

Summary of Changes

  • added 6 new rebuses with hints

@coveralls
Copy link

coveralls commented Feb 4, 2019

Pull Request Test Coverage Report for Build 440

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 86.617%

Totals Coverage Status
Change from base Build 435: 0.0%
Covered Lines: 154
Relevant Lines: 175

💛 - Coveralls

Copy link
Owner

@ollelauribostrom ollelauribostrom left a comment

Choose a reason for hiding this comment

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

Looks good, thanks! Added a comment about a minor nit that needs to be addressed before I can merge this 🙂

{
symbols: ['🍇', '-', '(🐒🐒🐒)', '+', '🐜'],
words: ['grant'],
hint: "Break it up. Do the subtraction then add the addition to the end"
Copy link
Owner

Choose a reason for hiding this comment

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

Could you change " to ' on line 782 to make the linter happy? 🙂

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It's in 👍

@ollelauribostrom
Copy link
Owner

Nice, thank you! 🎉

@ollelauribostrom ollelauribostrom merged commit 994af39 into ollelauribostrom:master Feb 6, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants