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

Translate Flag, Flags etc at CloudBuild page #92

Merged
merged 13 commits into from
Feb 13, 2025

Conversation

HThuren
Copy link
Contributor

@HThuren HThuren commented Feb 2, 2025

Here I make possible to translate at CloudBuildpage.

"Add flag": "Add flag",
"Flag": "Flag",
"Flags": "Flags",
"Missing flag": "Missing flag",
"Missing value": "Missing value",
"Value": "Value",

Maybe translate own:

@robustini
Copy link

IT

"Add flag": "Aggiungi flag",
"Flag": "Flag",
"Flags": "Flags",
"Missing flag": "Flag assente",
"Missing value": "Valore assente",
"Value": "Valore",

@Pat6874
Copy link

Pat6874 commented Feb 2, 2025 via email

@ulfhedlund
Copy link
Contributor

SE:
"Add flag": "Addera flagga",
"Flag": "Flagga",
"Flags": "Flaggor",
"Missing flag": "Flagga saknas",
"Missing value": "Värde saknas",
"Value": "Värde",

@HThuren
Copy link
Contributor Author

HThuren commented Feb 2, 2025

@pfeerick seems that building env missing some libs, in e2e-web and preview-web. No issue to compile locally

error message in e2e-web run:

Package libasound2 is a virtual package provided by:
liboss4-salsa-asound2 4.2-build2020-1ubuntu3
libasound2t64 1.2.11-1build2 (= 1.2.11-1build2)

Package libicu70 is not available, but is referred to by another package.
This may mean that the package is missing, has been obsoleted, or
is only available from another source

E: Package 'libasound2' has no installation candidate
E: Package 'libicu70' has no installation candidate
E: Unable to locate package libffi7
E: Unable to locate package libx264-163
Failed to install browsers
Error: Installation process exited with code: 100
Error: Process completed with exit code 1.

Copy link

codecov bot commented Feb 2, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 41.44%. Comparing base (6810861) to head (c247c9b).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #92      +/-   ##
==========================================
+ Coverage   39.53%   41.44%   +1.90%     
==========================================
  Files         103      103              
  Lines       12009    12010       +1     
  Branches      490      521      +31     
==========================================
+ Hits         4748     4977     +229     
+ Misses       7254     7024     -230     
- Partials        7        9       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@HThuren
Copy link
Contributor Author

HThuren commented Feb 2, 2025

I really don't understand the issue from Codecov Report, anyone ?

@pfeerick
Copy link
Member

pfeerick commented Feb 2, 2025

It is telling you that there are no unit tests covering those changes / testing that your changes don't break existing behaviour. If you have look at the messages in the "Files Changed" tab of the PR, you'll see comments by the CodeCov action as to which lines it is commenting on.

@Luznatural
Copy link

ES:

"Add flag": "Añadir flag",
"Flag": "Flag",
"Flags": "Flags",
"Missing flag": "Flag ausente",
"Missing value": "Valor ausente",
"Value": "Valor",

@zyren
Copy link

zyren commented Feb 3, 2025

ZH:


"Add flag": "添加标志",
"Flag": "标志",
"Flags": "标志",
"Missing flag": "缺失标志",
"Missing value": "缺失值",
"Value": "",

@pfeerick
Copy link
Member

pfeerick commented Feb 3, 2025

@pfeerick seems that building env missing some libs, in e2e-web and preview-web. No issue to compile locally

If you rebase on current main this should go away - it was due to upstream change in GitHub (changing from ubuntu-22.04 to ubuntu-24.04). You will then only hit the checks error related to this being done in a fork rather than in the buddy repo, but that is a known issue that is easy to see.

@freshollie
Copy link
Collaborator

Happy with this change. Annoying that preview runs don't work from forks yet. @pfeerick another option for your PR is to change the checkout step to "checkout PR" when doing pull requests.

@ParkerEde
Copy link

DE:
"Add flag": "Schalter hinzufügen",
"Flag": "Schalter",
"Flags": "Schalter",
"Missing flag": "Fehlender Schalter",
"Missing value": "Fehlender Wert",
"Value": "Wert",

@HThuren
Copy link
Contributor Author

HThuren commented Feb 6, 2025

@pfeerick seems that building env missing some libs, in e2e-web and preview-web. No issue to compile locally

If you rebase on current main this should go away - it was due to upstream change in GitHub (changing from ubuntu-22.04 to ubuntu-24.04). You will then only hit the checks error related to this being done in a fork rather than in the buddy repo, but that is a known issue that is easy to see.

I started with at fresh pull from main, and rebase are up to date, and stille have this issue. My commit only works with --no-verify, maybe that are a track to follow

@pfeerick
Copy link
Member

pfeerick commented Feb 6, 2025

It looks like something wasn't synced properly on your end still, as both your main and PR branches were one commit behind ;) Do you know to pull from the upstream branch, in order to keep your forks copy of main in sync with the upstream repo?

image

@pfeerick pfeerick force-pushed the translateFlag branch 2 times, most recently from e835da9 to 88a1994 Compare February 6, 2025 08:08
@pfeerick
Copy link
Member

pfeerick commented Feb 6, 2025

Please revert / stop pushing changes to src/shared/backend/__tests__/__snapshots__/sdcard.spec.ts.snap, as you are including invalid paths in those changes (hence why the tests is also failing, rather than just preview-web).

@HThuren
Copy link
Contributor Author

HThuren commented Feb 6, 2025

Please revert / stop pushing changes to src/shared/backend/__tests__/__snapshots__/sdcard.spec.ts.snap, as you are including invalid paths in those changes (hence why the tests is also failing, rather than just preview-web).

yes I struggle with that, now just deleted the unwanted file in pull, sorry you have to deal with this

@pfeerick
Copy link
Member

pfeerick commented Feb 6, 2025

When you commit changes, you should have the option of selecting what files to include in the commit... simply don't add / unselect the ones that are only relevant to your system. What software do you use to submit / track changes?

@HThuren
Copy link
Contributor Author

HThuren commented Feb 7, 2025

When you commit changes, you should have the option of selecting what files to include in the commit... simply don't add / unselect the ones that are only relevant to your system. What software do you use to submit / track changes?

I use git in cygwin64 terminal

... and yes I was stupid and used git add .

@pfeerick
Copy link
Member

pfeerick commented Feb 7, 2025

Yeah, done that myself (add all changes)... and it's fun (not!) trying to undo it sometimes, especially around submodules or irrelevant OS file changes.

@HThuren
Copy link
Contributor Author

HThuren commented Feb 8, 2025

@pfeerick sorry, but regarding codecov telling test are missing, but where to add more test, can't see other test constructions

@pfeerick
Copy link
Member

pfeerick commented Feb 9, 2025

I wouldn't worry about it... the only tests I care about atm are the ones done as part of the tests stage of the GitHub action... (and the action generally)... which are all passing.

 ✓ src/shared/backend/__tests__/firmware.spec.ts  (12 tests) 693ms
 ✓ src/shared/backend/__tests__/flash.spec.ts  (12 tests) [11](https://github.com/EdgeTX/buddy/actions/runs/13218759274/job/36901220073#step:5:12)56ms
 ✓ src/shared/backend/__tests__/cloudbuild.spec.ts  (2 tests) 169ms
 ✓ src/renderer/components/__tests__/VersionTargetForm.spec.tsx  (8 tests) 788ms
 ✓ src/shared/backend/__tests__/sdcard.spec.ts  (31 tests) 5114ms
 ✓ src/renderer/pages/flash/__tests__/FlashingWizard.spec.tsx  (7 tests) 2229ms
 ✓ src/renderer/components/devices/__tests__/DeviceSelector.spec.tsx  (7 tests) 980ms
 ✓ src/webworker/__test__/backend-worker.spec.ts  (5 tests) 410ms
 ✓ src/renderer/components/firmware/__tests__/FirmwareSummary.spec.tsx  (6 tests) 4[12](https://github.com/EdgeTX/buddy/actions/runs/13218759274/job/36901220073#step:5:13)ms
 ✓ src/renderer/compatibility/__tests__/CompatNoticeHandler.spec.tsx  (6 tests) 390ms
 ✓ src/renderer/components/firmware/__tests__/CopyUrlButton.spec.tsx  (4 tests) 233ms
 ✓ src/renderer/components/devices/__tests__/DeviceList.spec.tsx  (4 tests) 264ms
 ✓ src/renderer/components/firmware/__tests__/DownloadFirmwareButton.spec.tsx  (2 tests) 187ms
 ✓ src/renderer/components/flashing/__tests__/FlashButton.spec.tsx  (2 tests) 1[18](https://github.com/EdgeTX/buddy/actions/runs/13218759274/job/36901220073#step:5:19)ms
 ✓ src/renderer/components/devices/__tests__/DeviceSummary.spec.tsx  (2 tests) 338ms
 ✓ src/renderer/components/__tests__/WindowsNav.spec.tsx  (3 tests) 83ms
 ✓ src/renderer/components/__tests__/SettingsMenu.spec.tsx  (2 tests) 247ms
 ✓ src/renderer/components/__tests__/Markdown.spec.tsx  (1 test) 87ms

@kobakirill
Copy link
Contributor

kobakirill commented Feb 10, 2025

RU
"Add flag": "Добав флаг",
"Flag": "Флаг",
"Flags": "Флаги",
"Missing flag": "Отсут флаг",
"Missing value": "Отсут значение",
"Value": "Значение",

@kobakirill
Copy link
Contributor

BE
"Add flag": "Дадаць сцяг",
"Flag": "Сцяг",
"Flags": "Сцягі",
"Missing flag": "Адсут сцяг",
"Missing value": "Адсут значэнне",
"Value": "Значэнне",

@pfeerick
Copy link
Member

Since it looks like we've probably had all the translations we're going to, this can be merged when you say when ;)

@pfeerick
Copy link
Member

btw, you pushed src/shared/backend/__tests__/__snapshots__/sdcard.spec.ts.snap again 😇

@HThuren
Copy link
Contributor Author

HThuren commented Feb 13, 2025

@pfeerick believe the pull are ready to merge, please review

@HThuren
Copy link
Contributor Author

HThuren commented Feb 13, 2025

btw, you pushed src/shared/backend/__tests__/__snapshots__/sdcard.spec.ts.snap again 😇

yes , but now a copy from main branch

@pfeerick pfeerick enabled auto-merge (squash) February 13, 2025 23:27
@pfeerick
Copy link
Member

LGTM, and thanks for adding the tests for this... looks like some 41% of the codebase is covered by tests now :)

@pfeerick pfeerick merged commit 966d595 into EdgeTX:main Feb 13, 2025
14 of 15 checks passed
@HThuren HThuren deleted the translateFlag branch February 14, 2025 10:14
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.

10 participants