Skip to content
New issue

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

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test: check matthew #44

Merged
merged 2 commits into from
May 14, 2021
Merged

test: check matthew #44

merged 2 commits into from
May 14, 2021

Conversation

ghost
Copy link

@ghost ghost commented May 14, 2021

No description provided.

@ghost ghost requested a review from chanonroy May 14, 2021 13:52
@chanonroy chanonroy changed the title test/check matthew test: check matthew May 14, 2021
Copy link
Owner

@chanonroy chanonroy left a comment

Choose a reason for hiding this comment

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

Great addition! You have a bright future ahead of you!

@chanonroy chanonroy merged commit 20d88d2 into master May 14, 2021
@chanonroy chanonroy deleted the test/check-matthew branch May 14, 2021 13:56
Comment on lines +35 to +37
test('is Chanon equal to matthew', () => {
expect(isChanon('matthew')).toBe(false)
})
Copy link
Collaborator

Choose a reason for hiding this comment

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

Did we consider the capitalized version for this Hebrew-Biblical name ? how about the latin spelling ? 🤔

@husshegz
Copy link
Collaborator

@mattietea Can you look into this ?

@mattietea
Copy link

@mattietea Can you look into this ?

Oof, good shout. Created an issue here. Let's setup a meeting to comb through the backlog and prioritise ASAP. cc. @chanonroy

@chanonroy
Copy link
Owner

Thank you, gentleman.

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.

3 participants