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(demojify): Do not remove U+0023 # Number Sign #50

Merged
merged 3 commits into from
Sep 13, 2023

Conversation

mikesmithgh
Copy link
Contributor

👋 I noticed that the # number symbol is being removed from my bash code blocks. The demojify function is targeting # as if it is an emoji.

I see Demojify.lua is from https://github.com/buildthomas/Demojify/tree/master but I do not know if that project is still maintained. Let me know if you think I should open a PR to the upstream repo as well.

Thanks!

before fix

./panvimdoc.sh --project-name testing  --input-file "README.md" --toc false --demojify true && cat doc/testing.txt
SCRIPTS
pandoc --citeproc --shift-heading-level-by=0 --metadata=project:testing --metadata=vimversion: --metadata=toc:false --metadata=description: --metadata=dedupsubheadings:true --metadata=ignorerawblocks:true --metadata=docmapping:false --metadata=docmappingproject:true --metadata=treesitter:true --metadata=incrementheadinglevelby:0 --lua-filter=scripts/include-files.lua --lua-filter=scripts/skip-blocks.lua --data-dir=scripts/../lib --lua-filter=scripts/remove-emojis.lua -t scripts/panvimdoc.lua README.md -o doc/testing.txt
*testing.txt*                For                Last change: 2023 September 12

>bash
     this is a comment
    touch doc/${VIMDOC_PROJECT_NAME}.txt
    git commit -am "Add empty doc"
    git push
<

Generated by panvimdoc <https://github.com/kdheepak/panvimdoc>

vim:tw=78:ts=8:noet:ft=help:norl:

after fix

./panvimdoc.sh --project-name testing  --input-file "README.md" --toc false --demojify true && cat doc/testing.txt
SCRIPTS
pandoc --citeproc --shift-heading-level-by=0 --metadata=project:testing --metadata=vimversion: --metadata=toc:false --metadata=description: --metadata=dedupsubheadings:true --metadata=ignorerawblocks:true --metadata=docmapping:false --metadata=docmappingproject:true --metadata=treesitter:true --metadata=incrementheadinglevelby:0 --lua-filter=scripts/include-files.lua --lua-filter=scripts/skip-blocks.lua --data-dir=scripts/../lib --lua-filter=scripts/remove-emojis.lua -t scripts/panvimdoc.lua README.md -o doc/testing.txt
*testing.txt*                For                Last change: 2023 September 12

>bash
    # this is a comment
    touch doc/${VIMDOC_PROJECT_NAME}.txt
    git commit -am "Add empty doc"
    git push
<

Generated by panvimdoc <https://github.com/kdheepak/panvimdoc>

vim:tw=78:ts=8:noet:ft=help:norl:

@kdheepak
Copy link
Owner

Thanks for the PR! I didn’t realize that wasn’t maintained.

Would you be able to add a test for this? If not, I can add one later and then merge.

@mikesmithgh
Copy link
Contributor Author

Thanks for the PR! I didn’t realize that wasn’t maintained.

Would you be able to add a test for this? If not, I can add one later and then merge.

Sure I'll give it a shot

@mikesmithgh
Copy link
Contributor Author

@kdheepak Added tests 👍

@kdheepak
Copy link
Owner

Awesome! Haven't had a chance to look at the PR yet but thanks for doing this!

@kdheepak
Copy link
Owner

That looks great! Thanks again for submitting a PR! I think we should also upstream this fix, I bet the author would appreciate it.

Would you like to do it? I don't want to give you more work to do but also want you to get credit for your fix :)

@kdheepak kdheepak merged commit 9422960 into kdheepak:main Sep 13, 2023
1 check passed
@mikesmithgh
Copy link
Contributor Author

done and done buildthomas/Demojify#4 🙂 thanks!

@kdheepak
Copy link
Owner

Awesome! Thank you!

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