Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add write_reference_line_numbers option #322

Merged

Conversation

johantell
Copy link
Contributor

Background:
Since even the smallest changes in files can cause a cascade of changed line numbers in po/pot files we're often seeing larger amounts of changes in commits causing the need to re-extract than needed.

By adding the write_reference_line_numbers option developers has a way to decide for themselves whether they want to include them or not.

Fixes #321

@johantell johantell force-pushed the allow_pruning_of_line_numbers branch from 2d8b661 to d13f8d2 Compare October 7, 2022 19:28
Background:
Since even the smallest changes in files can cause a cascade of
changed line numbers in po/pot files we're often seeing larger amounts
of changes in commits causing the need to re-extract than needed.

By adding the `write_reference_line_numbers` option developers has a way
to decide for themselves whether they want to include them or not.

Fixes elixir-gettext#321
@johantell johantell force-pushed the allow_pruning_of_line_numbers branch from d13f8d2 to c7ce1c1 Compare October 7, 2022 19:29
@johantell johantell marked this pull request as draft October 7, 2022 19:32
@johantell johantell marked this pull request as ready for review October 7, 2022 19:34
lib/gettext.ex Outdated Show resolved Hide resolved
Co-authored-by: Szymon Jeż <35540+szymon-jez@users.noreply.github.com>
@josevalim josevalim merged commit 87a63df into elixir-gettext:main Oct 19, 2022
@josevalim
Copy link
Contributor

💚 💙 💜 💛 ❤️

@johantell johantell deleted the allow_pruning_of_line_numbers branch October 22, 2022 08:06
ravensiris pushed a commit to ravensiris/gettext that referenced this pull request Aug 3, 2023
Background:
Since even the smallest changes in files can cause a cascade of
changed line numbers in po/pot files we're often seeing larger amounts
of changes in commits causing the need to re-extract than needed.

By adding the `write_reference_line_numbers` option developers has a way
to decide for themselves whether they want to include them or not.

Fixes elixir-gettext#321
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.

Possibility to remove line-numbers from extraction metadata
4 participants