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

Add extension points to preprocess2 #4394

Open
jelovirt opened this issue Feb 11, 2024 · 0 comments · May be fixed by #4395
Open

Add extension points to preprocess2 #4394

jelovirt opened this issue Feb 11, 2024 · 0 comments · May be fixed by #4395
Labels
feature New feature or request preprocess2

Comments

@jelovirt
Copy link
Member

jelovirt commented Feb 11, 2024

Description

Add Ant target extension points to preprocess2:

  • preprocess2.topics.pre
  • preprocess2.topics.post
  • preprocess2.maps.pre
  • preprocess2.maps.post

The target depends list will be

{depend.preprocess.pre},
preprocess2.init,
ditaval-merge,

{depend.preprocess2.maps.pre},
preprocess2.maps,
{depend.preprocess2.maps.post},
{depend.preprocess2.topics.pre},
preprocess2.topics,
{depend.preprocess2.topics.post},

map-clean-map,
clean-preprocess,
copy-files2,
preprocess2.post,
{depend.preprocess.post}

Possible Solution

Allow plug-ins to extend preprocess2 from "safe" points which are likely not be changed in preprocess2.

Potential Alternatives

Plug-ins can fork preprocess2.

@jelovirt jelovirt added feature New feature or request preprocess2 labels Feb 11, 2024
@jelovirt jelovirt changed the title Add extension points to preprcess2 Add extension points to preprocess2 Feb 11, 2024
@jelovirt jelovirt linked a pull request Feb 11, 2024 that will close this issue
@jelovirt jelovirt linked a pull request Feb 11, 2024 that will close this issue
@robander robander assigned robander and unassigned robander Apr 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request preprocess2
Projects
Status: In Progress
Development

Successfully merging a pull request may close this issue.

2 participants