Skip to content

refactor(uk-ua.json): Translations#2033

Closed
PavloPogonets wants to merge 3 commits into
Acode-Foundation:mainfrom
PavloPogonets:patch-3
Closed

refactor(uk-ua.json): Translations#2033
PavloPogonets wants to merge 3 commits into
Acode-Foundation:mainfrom
PavloPogonets:patch-3

Conversation

@PavloPogonets

Copy link
Copy Markdown
Contributor

No description provided.

@github-actions github-actions Bot added the translations Anything related to Translations Whether a Issue or PR label Apr 14, 2026
@greptile-apps

greptile-apps Bot commented Apr 14, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR adds Ukrainian translations for ~35 previously untranslated strings in src/lang/uk-ua.json, covering LSP install method labels, settings category names, and settings info descriptions.

Two minor consistency issues were found: lsp-install-method-npm ("npm пакет") and lsp-install-method-pip ("пакет pip") use opposite word orders, and three adjacent keys (lsp-install-method-cargo, lsp-install-method-manual, lsp-install-method-title) were left in English while their neighbours were translated.

Confidence Score: 4/5

  • Safe to merge with minor consistency improvements recommended before landing.
  • All findings are P2 (style/consistency). The inconsistent word order and three untranslated strings in the same section are worth addressing for a clean translation, but they do not block functionality or correctness.
  • src/lang/uk-ua.json — specifically the lsp-install-method-* group (lines 556–562)

Important Files Changed

Filename Overview
src/lang/uk-ua.json Translates ~35 strings from English to Ukrainian across LSP install methods, settings categories, and settings info entries. Minor issues: inconsistent word order between lsp-install-method-npm/lsp-install-method-pip, and three adjacent lsp-install-method-* keys left untranslated.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[uk-ua.json] --> B[LSP strings]
    A --> C[Settings categories]
    A --> D[Settings info]

    B --> B1["✅ lsp-install-method-apk → Пакет APK"]
    B --> B2["⚠️ lsp-install-method-cargo → Cargo crate (untranslated)"]
    B --> B3["⚠️ lsp-install-method-manual → Manual binary (untranslated)"]
    B --> B4["✅ lsp-install-method-npm → npm пакет"]
    B --> B5["✅ lsp-install-method-pip → пакет pip (word order differs)"]
    B --> B6["✅ lsp-install-method-shell → Користувацька оболонка"]
    B --> B7["⚠️ lsp-install-method-title → Install method (untranslated)"]
    B --> B8["✅ lsp-installed → Встановлено"]
    B --> B9["✅ lsp-status-not-installed → Не встановлено"]
    B --> B10["✅ lsp-remove-custom-server → Видалити користувацький сервер"]

    C --> C1["✅ 22 settings-category-* strings translated"]
    D --> D1["✅ settings-info-app-vibrate-on-tap translated"]
    D --> D2["✅ settings-info-editor-line-numbers translated"]
    D --> D3["✅ settings-info-main-edit-settings translated"]
Loading

Reviews (1): Last reviewed commit: "refactor(uk-ua.json): Translations" | Re-trigger Greptile

Comment thread src/lang/uk-ua.json Outdated
Comment thread src/lang/uk-ua.json Outdated
Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com>
@UnschooledGamer

Copy link
Copy Markdown
Member

@PavloPogonets Hey, is the Pull Request Ready, as I can see some strings untranslated?

@UnschooledGamer UnschooledGamer added community WIP Work in progress labels Apr 15, 2026
@UnschooledGamer UnschooledGamer moved this from Backlog to In progress in The Code Board - Acode Apr 15, 2026
@PavloPogonets

Copy link
Copy Markdown
Contributor Author

I did a final translation, but it seems to me that the line "Cargo crate" is not translated into Ukrainian.

@UnschooledGamer

Copy link
Copy Markdown
Member

I did a final translation, but it seems to me that the line "Cargo crate" is not translated into Ukrainian.

Yes, that's right.

@UnschooledGamer

Copy link
Copy Markdown
Member

@PavloPogonets, is this mergeable, or may I convert this into a Draft Pull Request?

@PavloPogonets

Copy link
Copy Markdown
Contributor Author

I think it might be better to close it and I'll make a new translation because there are some errors here, there will probably be fewer problems.

@UnschooledGamer

Copy link
Copy Markdown
Member

I think it might be better to close it and I'll make a new translation because there are some errors here, there will probably be fewer problems.

@PavloPogonets Alright, I'll close it.

@github-project-automation github-project-automation Bot moved this from In progress to Done in The Code Board - Acode May 2, 2026
@UnschooledGamer UnschooledGamer removed the WIP Work in progress label May 2, 2026
@UnschooledGamer UnschooledGamer moved this from Done to 🫡 Done by Community in The Code Board - Acode May 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community translations Anything related to Translations Whether a Issue or PR

Projects

Status: 🫡 Done by Community

Development

Successfully merging this pull request may close these issues.

2 participants