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

[ucd/category] Fix typo in GeneralCategory::is_symbol() #60

Merged
merged 1 commit into from
Jul 27, 2017

Conversation

calum
Copy link

@calum calum commented Jul 27, 2017

While I was working on #43 I found this typo which was causing one of my tests to fail.

I haven't finished #43 yet but I figured I may as well fix the typo asap.

My first ever PR! :')

@calum calum closed this Jul 27, 2017
@calum
Copy link
Author

calum commented Jul 27, 2017

I closed this because I have made another pull request which includes this change: #61

@behnam behnam reopened this Jul 27, 2017
@behnam behnam changed the title fixed typo in GeneralCategory implentation for is_symbol() [ucd/category] Fix typo in GeneralCategory::is_symbol() Jul 27, 2017
@behnam behnam merged commit 98d9c16 into open-i18n:master Jul 27, 2017
behnam added a commit that referenced this pull request Jul 27, 2017
The coverage test makes sure there's no overlap between major
categories, except the case of `Cased_Letter` and `Letter`.

The test can catch bugs like the one fixed in
<#60>.
behnam added a commit that referenced this pull request Jul 28, 2017
The coverage test makes sure there's no overlap between major
categories, except the case of `Cased_Letter` and `Letter`.

The test can catch bugs like the one fixed in
<#60>.
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