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

fix: patchDocument, looks for namspaces more carefully over whole doc… #2943

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

13W
Copy link

@13W 13W commented Jan 24, 2025

…ument

added support of different placeholder brackets, default {{: }}

extended inputDataType to accept JSZip

Vladimir Bulyga and others added 3 commits January 24, 2025 17:55
…ument

added support of different placeholder brackets, default `{{`: `}}`

extended inputDataType to accept JSZip
@dolanmiu
Copy link
Owner

Can you add a test to your PR?

@13W
Copy link
Author

13W commented Feb 5, 2025

sorry guys, i don't think that i have enough time to write tests... and by specific of my work - i have to fix few bugs in your library to satisfy our requirements...
it would be good if someone fix the problems that i faced with, within your workflow...
i can explain each fix that i did if you wish

@dolanmiu
Copy link
Owner

dolanmiu commented Feb 5, 2025

Hi @13W

Sure, yes, can you explain each fix? Maybe it can be a comment in the code

added parameter `recursive`(= true by default) to be able to disable
this behavior
@dolanmiu dolanmiu self-requested a review February 8, 2025 15:05
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