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
Remove PDF2 AXF and RenderX in favour of separate repo #3807
Conversation
ae56642
to
ed410b1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested with a fresh Git worktree , both external plug-ins are installed correctly and distribution build succeeds. ✅
Should we also remove the corresponding lines from the Git ignore file? |
There are still traces of XEP in org.dita.pdf2/resource/messages.xml. Would it make sense to move these to the external plug-in as well, or are there reasons to keep them in the core PDF plug-in? |
ed410b1
to
4ce317f
Compare
Signed-off-by: Jarno Elovirta <jarno@elovirta.com>
Signed-off-by: Jarno Elovirta <jarno@elovirta.com>
4ce317f
to
505e1b1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
With the migration of the XEP messages from org.dita.pdf2
to dita-ot/org.dita.pdf2.xep@42b8628 for plug-in v3.6.2, I think this is good to go. ✅
Description
Remove PDF2 AXF and RenderX plug-ins and install them from dita-ot/org.dita.pdf2.axf and dita-ot/org.dita.pdf2.xep into distribution package.
Motivation and Context
PDF2 AXF and RenderX plug-ins have been moved to their own repos.
How Has This Been Tested?
Distribution build works.
Type of Changes
Documentation and Compatibility
Document that PDF2 AXF and RenderX related bugs should be reported to https://github.com/dita-ot/org.dita.pdf2.axf/issues and https://github.com/dita-ot/org.dita.pdf2.xep/issues, respectively.