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

PO(T) Export, for message extraction #45

Merged

Conversation

TropicalPenguin
Copy link
Contributor

Permits the writing of message extractors, using gotext. With round-tripping support to update existing translations (while preserving contextual details like copyright notices & source references).

@leonelquinteros
Copy link
Owner

@TropicalPenguin Thanks for your contribution.

Please note that there are some conflicts after I merged a previous PR I had pending.
Can you also review the changes included in the last PR ( #44 ) and adapt your feature to the new code? I think we can benefit from both together.

@TropicalPenguin
Copy link
Contributor Author

Yeah, no problem; got a few more improvements to make anyway.

Allows stable round-tripping of PO(T) data across multiple message extraction passes.
Avoids canonicalisation of header keys, while preserving other semantics.
@TropicalPenguin
Copy link
Contributor Author

I think that should do it 😄

@leonelquinteros leonelquinteros merged commit 327b393 into leonelquinteros:master Oct 5, 2020
@leonelquinteros
Copy link
Owner

Thank you @TropicalPenguin !

@TropicalPenguin TropicalPenguin deleted the master-potexport branch October 5, 2020 16:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants