-
Notifications
You must be signed in to change notification settings - Fork 7
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
Add translit ipa #31
Add translit ipa #31
Conversation
Pull Request Test Coverage Report for Build 126
💛 - Coveralls |
@ManjotS @navdeepsinghkhalsa can we merge & close this out |
@navdeepsinghkhalsa I am confident enough in it (it will need adjustments in the future just as english does) can we merge? |
@@ -1,7 +1,7 @@ | |||
{ | |||
"name": "anvaad-js", | |||
"version": "1.0.9", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This will be done by npm version
when releasing
Could you also please resolve conflicts? |
@@ -3,6 +3,9 @@ const translit = require('../translit'); | |||
describe('translit', () => { | |||
it('Should return transliteration of Gurmukhi', () => { | |||
expect(JSON.stringify(translit('CMqu ] ijau jwnhu iqau rwKu hir pRB qyirAw ] kyqy gnau AsMK Avgx myirAw ] AsMK Avgx Kqy Pyry inqpRiq sd BUlIAY ] moh mgn ibkrwl mwieAw qau pRswdI GUlIAY ] lUk krq ibkwr ibKVy pRB nyr hU qy nyirAw ] ibnvMiq nwnk dieAw Dwrhu kwiF Bvjl PyirAw ]1]', 'all'))) | |||
.toBe(JSON.stringify({ english: 'chha(n)t || jiau jaanahu tiau raakh har prabh teriaa || kete ganau asa(n)kh avagan meriaa || asa(n)kh avagan khate fere nitaprat sadh bhooleeaai || moh magan bikaraal maiaa tau prasaadhee ghooleeaai || look karat bikaar bikhaRe prabh ner hoo te neriaa || binava(n)t naanak dhiaa dhaarahu kaadd bhavajal feriaa ||1||' })); | |||
.toBe(JSON.stringify({ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is one test enough?
@ManjotS please follow up here |
* English updates (#30) * Added rules for english cleanup, remove extra vowels in specific words guramukh, gurabani, guradhev, gurasikh, manamukh * words that start with ooRa like updesh should be updesh not aupdesh * [WIP] Hindi Translit (#19) * Initial Commit * Add better support * register hindi in translit.js * Add some more fixes to hindi * rename to devnagri * updated to add all characters required Some still need devnagri equivalent * Update devnagri.js * Update devnagri.js * Update devnagri.js * Update tests * remove prettier * ignore broken tests * Update LICENSE.md (#32) * Update package.json * Adds more tests for translit function; corrects output in comments of code * Removes changes from .eslintignore and .gitignore
This was actually a part of Guaravjeet Singh's work...
…On Tue, Feb 5, 2019 at 5:33 AM Navdeep Singh ***@***.***> wrote:
Merged #31 <#31> into
master.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#31 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAnXFWpq6K-z8uA39mJufz7ruQnbbxoyks5vKYgbgaJpZM4Wn8db>
.
|
No description provided.