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

Add "nòng" #24

Merged
merged 1 commit into from
Jun 17, 2023
Merged

Add "nòng" #24

merged 1 commit into from
Jun 17, 2023

Conversation

ngoclong19
Copy link
Contributor

As in "nòng cốt", "nòng súng", "nòng nọc".

I sort the file using this command in Linux:
LC_ALL=C sort vi-DauMoi.dic -u

Copy link
Owner

@1ec5 1ec5 left a comment

Choose a reason for hiding this comment

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

Thank you!

@1ec5 1ec5 merged commit 39cc647 into 1ec5:master Jun 17, 2023
@ngoclong19
Copy link
Contributor Author

You are welcome!

I'm concerned that this add-on may be outdated. Do you think we could set up an automated release process to ensure that it's always up to date?
https://addons.mozilla.org/firefox/addon/vietnamese-dictionary/

@1ec5
Copy link
Owner

1ec5 commented Jun 17, 2023

Yes, it is outdated. There’s a request in #23 to make a new release, but it’s been so long since the last one (almost a decade) that I’m not sure if the build scripts will be compatible with Mozilla’s signing process and anything new that has happened in LibreOffice. The Mozilla build script is based on AVIM’s, which have been updated in the meantime. I’d appreciate any help in modernizing the build workflow; otherwise, I’ll try to find time to get it back into shape, but I can’t guarantee when it’ll come out unfortunately.

@ngoclong19
Copy link
Contributor Author

I will create a proof of concept for a new build workflow by this year, will get back to you if I come up with anything meaningful.

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