Comments (8)
@whatyouhide Should we directly do a patch release? I think this might impact a few users...
from gettext.
Sorry for the confusion and thanks for the fix! I have just successfully tested the main branch in my application.
from gettext.
@krns Do you get a changed file running the extract task? If yes: what is different?
Edit: nevermind, read the issue description again and i can answer my own question.
from gettext.
Some clarifications because my problem description might be a bit misleading:
When I talk about "same result", I mean that the warning remains. The newly extracted POT files are actually slightly different from the POT file extracted with gettext version 0.20.0:
## This file is a PO Template file.
##
## "msgid"s here are often extracted from source code.
-## Add new translations manually only if they're dynamic
-## translations that can't be statically extracted.
+## Add new message manually only if they're dynamic
+## message that can't be statically extracted.
##
## Run "mix gettext.extract" to bring this file up to
## date. Leave "msgstr"s empty as changing them here has no
from gettext.
@krns thanks for the report!
First of all, I don't think that comment change is necessary, and it's going to cause a bunch of unnecessary changes in people's projects. @maennchen, I'll revert that to the previous comment.
@krns, do you happen to have a minimal reproducing project that I could use to debug this?
from gettext.
@whatyouhide That change was part of an effort to only use the term „message“ and not „translation“ in the whole codebase. This should make understanding the documentation simpler, since we always use the same term for messages.
„message“ is the name that gettext itself uses.
In this case, I personally think consistency is worth more than a tiny change produced in a file. I think it would be reasonable for a user of this library to update the .pot?
files before expecting them to be up to date.
WDYT?
from gettext.
This is a minimal reproducing project: https://github.com/krns/elixir-gettext-0.21.0-test
Steps to reproduce:
- extract via
mix gettext.extract
- check if up to date via
mix gettext.extract --check-up-to-date
from gettext.
@whatyouhide We miss-interpreted the issue:
The new comment is not forced onto users at all. It will only matter for new .pot
files. The issue instead comes from #335 (write_reference_comments
=> false
)
from gettext.
Related Issues (20)
- Flaky test in `gettext.extract`
- Duplicate Filename in Reference when `:write_reference_line_numbers` is set to `false`
- Gettext.PluralFormError for plural form "1" in "ja" locale HOT 9
- `gettext.merge` FunctionClauseError HOT 4
- Module is not loaded because :nofile HOT 4
- Mention file path in plural forms deprecation warning HOT 4
- Plural Forms warning occurs in newly generated language file HOT 2
- bump a new version for #359 HOT 1
- Duplicate msgid with singular and plural form HOT 5
- Running `mix gettext.extract` doesn't extract new messages with Elixir 1.15 HOT 2
- Retain custom flags during merge HOT 3
- Interpolation option set but not working HOT 4
- Duplicate references in POT files and warnings about redefining modules HOT 5
- compile depend excoveralls - origin/httpc failed HOT 1
- Add `Gettext.example` macro HOT 6
- If changes to `.po` file are discarded (accidentally), they're not added back HOT 17
- Allow to transform messages at compile time HOT 3
- `expo.msguniq` merges translations with different plurals HOT 4
- Locale changes between static mount and liveview HOT 1
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from gettext.