Skip to content

Conversation

arturz
Copy link
Collaborator

@arturz arturz commented Mar 7, 2025

Closes #107

@arturz arturz self-assigned this Mar 7, 2025
@arturz arturz requested a review from szsoppa March 7, 2025 11:43
@arturz arturz force-pushed the 107-remove-dependency-on-specific-gettext-version branch 8 times, most recently from 94c4ea9 to 41561fb Compare March 12, 2025 09:13
@arturz arturz force-pushed the 107-remove-dependency-on-specific-gettext-version branch from 41561fb to a26baa5 Compare March 12, 2025 09:15
- Gettext 0.26.0+
- PostgreSQL 15+ or SQLite 3.31.0+

## Installation
Copy link
Collaborator Author

@arturz arturz Mar 12, 2025

Choose a reason for hiding this comment

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

Remove note about specific gettext version requirement under the Installation section

alias Kanta.Utils.Compilation
defmodule Kanta.Gettext.Adapter do
@moduledoc """
Kanta adapter used in *gettext functions from Kanta.Gettext.Macros.
Copy link
Contributor

Choose a reason for hiding this comment

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

Just wondering if *gettext shouldn't be gettext maybe?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

No, because there are a bunch of functions ending with gettext, such as ngettext, pgettext, dgettext, etc.

@arturz arturz requested a review from vi0dine March 13, 2025 12:12
Copy link
Contributor

@vi0dine vi0dine left a comment

Choose a reason for hiding this comment

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

Lgtm

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