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

Remove gettext from project #291

Merged
merged 1 commit into from
Jan 8, 2025
Merged

Remove gettext from project #291

merged 1 commit into from
Jan 8, 2025

Conversation

ikagod
Copy link
Collaborator

@ikagod ikagod commented Jan 8, 2025

Gettext is unnecessary as ikabot is a relatively small project. We do not have enough resources to add different locales and even if we did, it would require a lot of resources to maintain those localizations when new modules are written in english.

To add to this, gettext wasn't even used across the entire project, some modules used it for all strings, some modules used it only for some strings, and not other and some modules didn't use it at all.

All in all, I think it's safe to assume ikabot is better off without gettext

Gettext is unnecessary as ikabot is a relatively small project. We do not have enough resources to add different locales and even if we did, it would require a lot of resources to maintain those localizations when new modules are written in english.

To add to this, gettext wasn't even used across the entire project, some modules used it for all strings, some modules used it only for some strings, and not other and some modules didn't use it at all.

All in all, I think it's safe to assume ikabot is better off without gettext
@ikagod ikagod added the release notes Items are likely to be highlighted in the release notes. label Jan 8, 2025
@ikagod ikagod added this to the 7.1.0 milestone Jan 8, 2025
@ikagod ikagod added the enhancement New feature or request label Jan 8, 2025
@ikagod ikagod merged commit ad180a1 into master Jan 8, 2025
10 checks passed
@ikagod ikagod deleted the remove-gettext branch January 8, 2025 18:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request release notes Items are likely to be highlighted in the release notes.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant