Skip to content

Update FrenchLayout.js #59

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

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

Conversation

SamKhachatryan
Copy link

@SamKhachatryan SamKhachatryan commented Dec 23, 2024

French layout unneccesary repeating letters removal.

P.S. this package is deeply integrated in our current ecosystem, and we don't plan to change it. I kindly ask to approve the PR if it seems okay, when you'll have time. Also we can discuss maintainance rules if you don't plan to work on this anymore.

French layout unneccesary repeating letters removal.
@reganlawton
Copy link
Collaborator

@SamKhachatryan I'll review this tomorrow, looks fine on a quick check. I'm just going for a Xmas eve dinner and just want to run it to confirm it's all good 🤞

@SamKhachatryan
Copy link
Author

Sure, take your time, thanks a lot.

@SamKhachatryan
Copy link
Author

SamKhachatryan commented Dec 24, 2024

Btw seems like I made a mistake, changing inside lib instead of src

@reganlawton
Copy link
Collaborator

@SamKhachatryan I'm back at work and have more availability but in discuss to your comment on the last about lib instead of src, were you going to make that change or wanting me too, with a new commit to the main repo?

I'm been waiting for the commit and now that I'm back I thought I'd touch base with you to try get this moving.

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