Skip to content

fix(ruby): fix localization and scripts conversion#1995

Merged
imobachgs merged 7 commits intomasterfrom
fix-converter
Feb 14, 2025
Merged

fix(ruby): fix localization and scripts conversion#1995
imobachgs merged 7 commits intomasterfrom
fix-converter

Conversation

@imobachgs
Copy link
Contributor

@imobachgs imobachgs commented Feb 13, 2025

Problem

The conversion of the localization and scripts sections has several problems:

This PR include some changes from #1951 and #1946.

Solution

  • Fix the handling of the scripts file name and location.
  • Fix the name of the localization setting ("localization" instead of "l10n").

Additionally, use "pretty" JSON when exporting the converted profile.

Testing

  • Added a new unit test

Copy link
Contributor

@joseivanlopez joseivanlopez left a comment

Choose a reason for hiding this comment

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

LGTM

@imobachgs imobachgs merged commit ef358c8 into master Feb 14, 2025
7 checks passed
@imobachgs imobachgs deleted the fix-converter branch February 14, 2025 08:58
@imobachgs imobachgs mentioned this pull request Feb 26, 2025
imobachgs added a commit that referenced this pull request Feb 26, 2025
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.

3 participants