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

feat: add ability to define empty tag format #110

Merged
merged 1 commit into from
Nov 28, 2022

Conversation

warlof
Copy link
Contributor

@warlof warlof commented Nov 4, 2022

add a new setting called useSelfClosingTags which allow to define how empty tag must be write while generating new XLIFF synchronized files.

its default value is true meaning empty tag will use self closing format (<note/>) - as it was the case prior this change.

when set to false, empty tag will be write using opening and closing format (<note></note>).

as an example, AL Compiler is using the opening and closing format instead the self-closing one.
using this extension current version with AL project is then generating excessive git diff for small changes since all empty tags are updated to the self-closing format.

references :

Closes #97

add a new setting called `useSelfClosingTags` which allow to define how empty tag must be write while generating new XLIFF synchronized files.

its default value is `true` meaning empty tag will use self closing format (<note/>) - as it was the case prior this change.

when set to `false`, empty tag will be write using opening and closing format (<note></note>).

references :
- oozcitak/xmlbuilder-js#65
- https://github.com/oozcitak/xmlbuilder-js/wiki#converting-to-string

Closes rvanbekkum#97
@rvanbekkum rvanbekkum self-requested a review November 7, 2022 13:32
@rvanbekkum
Copy link
Owner

Thank you for the PR. I have already checked the contents and it looks good. I will probably merge/release it next weekend (after Directions EMEA in Hamburg).

@warlof
Copy link
Contributor Author

warlof commented Nov 23, 2022

Hello there, just a small bump about this PR

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.

2 participants