Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Add test for monetary formatting (USD) #46

Merged
merged 1 commit into from Aug 22, 2012

Conversation

Projects
None yet
2 participants
Contributor

jugglinmike commented Aug 22, 2012

After writing, I realized that this might be a little awkward for non-US participants. Thought I'd submit and see what you thought.

Owner

rmurphey commented Aug 22, 2012

It's still a regular expression, so I'm all for it. Thanks for this :) Any chance you want to add answers here too?

rmurphey added a commit that referenced this pull request Aug 22, 2012

Merge pull request #46 from jugglinmike/money-format
Add test for monetary formatting (USD)

@rmurphey rmurphey merged commit e73fd5d into rmurphey:master Aug 22, 2012

@jugglinmike jugglinmike referenced this pull request in rmurphey/js-assessment-answers Aug 22, 2012

Merged

Add implementation of "isUSD" method #3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment