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

Add missing character 'ی' #11

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

hsharghi
Copy link

@hsharghi hsharghi commented Dec 9, 2019

No description provided.

Copy link

@siemensikkema siemensikkema left a comment

Choose a reason for hiding this comment

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

I'll trust that this is the right classification. Thanks!

Copy link

@siemensikkema siemensikkema left a comment

Choose a reason for hiding this comment

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

The change is fine but CI is breaking. Could you try and update .circleci/config.yml and change the line xcode: "9.0" to `xcode: "9.2"?

@BrettRToomey
Copy link
Contributor

This is surprisingly hard to look up, but it seems like it's this here.

@BrettRToomey
Copy link
Contributor

Looks like it can be both Î and Y

@hsharghi
Copy link
Author

Looks like it can be both Î and Y

The character 'ي' was properly set in replacement dictionary. But the character 'ی' was not exist in the dictionary at all, and the latter character was being replaced with '-' when slugify() the word.

@hsharghi
Copy link
Author

The change is fine but CI is breaking. Could you try and update .circleci/config.yml and change the line xcode: "9.0" to `xcode: "9.2"?

Should I change the config.yml file?

@siemensikkema
Copy link

@hsharghi yes please, that should fix the CI build

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.

None yet

3 participants