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

chore: replace easygettext with scripts from vue3 gettext #10389

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

Conversation

LukasHirt
Copy link
Contributor

Description

  • use vue3 gettext to extract and compile translations
  • introduce gettext config which configures extraction and compilation
  • drop LC_MESSAGES from path of translation files (needed due to configuration of extraction and compilation where we cannot specify path after locale)

Related Issue

Motivation and Context

Get rid of dep alerts

How Has This Been Tested?

  • test environment: macos x64
  • test case 1: makefile

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Technical debt
  • Tests
  • Documentation
  • Maintenance (e.g. dependency updates or tooling)

Copy link

update-docs bot commented Jan 22, 2024

Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes.

@LukasHirt
Copy link
Contributor Author

@JammingBen @dschmidt please let me know how would you like to approach changelog here

@dschmidt
Copy link
Member

no change log pretty much irrelevant for users and external developers

@LukasHirt LukasHirt added Status:Needs-Review Needs review from a maintainer and removed Status:Needs-Info labels Jan 23, 2024
Copy link

sonarcloud bot commented Jan 23, 2024

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Category:Technical Technical ehancements Status:Needs-Review Needs review from a maintainer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Get rid of easygettext and use vue3-gettext instead
2 participants