Skip to content

feat: update langs - #93

Merged
gbprod merged 1 commit into
mainfrom
feat/update-lang-3
Aug 25, 2026
Merged

feat: update langs#93
gbprod merged 1 commit into
mainfrom
feat/update-lang-3

Conversation

@gbprod

@gbprod gbprod commented Aug 25, 2026

Copy link
Copy Markdown
Owner

No description provided.

Copilot AI lite review requested due to automatic review settings August 25, 2026 13:18

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟡 Changes recommended

Regenerate the checked-in grammar artifacts and add tests for the new localized variants.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Pull request overview

Adds Bulgarian and Turkish localized Git commit-message variants.

Changes:

  • Adds a Bulgarian generated-comment separator.
  • Adds Turkish double-quoted branch status variants.
File summaries
File Review findings
grammar.js Generated grammar artifacts are stale, and regression coverage is missing for the new alternatives.
Review details

Suppressed comments (3)

grammar.js:181

  • The existing generated-comments corpus has Bulgarian separator fixtures, but none contains this newly accepted shortened line. Please add a corpus case in test/corpus/generated_comments.txt so this exact localization is verified rather than leaving the new grammar branch untested.
            "# Въведете съобщението за подаване на промѐните.  Редовете, които",

grammar.js:410

  • The generated-comments corpus currently exercises the single-quoted Turkish up-to-date form, but not this double-quoted form. Please add an input/expected-tree case for the new literal so a future regeneration or edit cannot silently break it.
        seq("Dalınız \"", $.branch, "\" ile güncel."),

grammar.js:498

  • The generated-comments corpus currently exercises the single-quoted Turkish ahead form, but not this comma/double-quoted variant. Please add an input/expected-tree case for the new literal so this newly accepted Git output is covered by the parser tests.
        seq("Dalınız, \"", $.branch, "\" dalından ", $.number, " işleme ileride."),
  • Files reviewed: 1/1 changed files
  • Comments generated: 2
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread grammar.js
Comment on lines +181 to 182
"# Въведете съобщението за подаване на промѐните. Редовете, които",
"# Въведете съобщението за подаване на промените. Редовете, които започват",
Comment thread grammar.js
Comment on lines +181 to 182
"# Въведете съобщението за подаване на промѐните. Редовете, които",
"# Въведете съобщението за подаване на промените. Редовете, които започват",
@gbprod
gbprod force-pushed the feat/update-lang-3 branch from 9383821 to db4a0e1 Compare August 25, 2026 13:29
@gbprod
gbprod merged commit a87878a into main Aug 25, 2026
3 checks passed
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.

2 participants