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 support for structurizr includes. #22

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

willson556
Copy link

Changes:

  • Adds a map of diagram types to preprocessors.
  • Adds a structurizr include preprocessor.

Resolves #11.

@alexander-bauer
Copy link

I've tried out using this branch tip, and it's working nicely for me. I'd love to see this merged to main. The code strikes me as in good shape, with these addendums (which might be outside of the scope of this PR):

  1. It might be handy to expose the plugin configuration to preprocessors, if it's not already available through the global environment. I could see this coming up with cases like user macros, or preprocessor logic that uses external tools.
  2. If there get to be more Preprocessors, then it might be handy for the preprocessor dict to contain a sequence of Preprocessors to be applied, rather than just one.

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.

Rendering structurizr graphic with includes fails
2 participants