Skip to content

Comments

Improved locales#34

Merged
spietras merged 1 commit intomainfrom
locales
Feb 10, 2026
Merged

Improved locales#34
spietras merged 1 commit intomainfrom
locales

Conversation

@spietras
Copy link
Member

No description provided.

@spietras spietras requested a review from Copilot February 10, 2026 18:54
@spietras spietras self-assigned this Feb 10, 2026
@spietras spietras added the cleanup Stay fresh label Feb 10, 2026
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR improves the localization setup by refining TypeScript type definitions and standardizing locale file metadata across the project's template files.

Changes:

  • Updated TypeScript declaration for .po module imports to use explicit import type syntax
  • Standardized metadata field ordering in both English and Polish locale template files
  • Changed Project-Id-Version to use the {{ appname }} template variable for dynamic app naming
  • Removed fixed POT-Creation-Date in favor of empty value, appropriate for template files

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.

File Description
src/src/common/localization/locales/types.d.ts Refactored import to use explicit import type syntax, aligning with codebase conventions
src/src/common/localization/locales/en.po.jinja Reordered metadata fields and updated Project-Id-Version to use template variable
src/src/common/localization/locales/pl.po.jinja Applied same metadata improvements as English locale for consistency

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@spietras spietras merged commit db371ce into main Feb 10, 2026
8 checks passed
@spietras spietras deleted the locales branch February 10, 2026 18:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cleanup Stay fresh

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant