-
Notifications
You must be signed in to change notification settings - Fork 93
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
Including C4-PlantUML locally with relative path #3
Comments
Looks like this is always going to the remote server (www.plantuml.com): |
I'm working on a feature to allow it to clone the C4-PlantUML repo locally. Hopefully it will be up this weekend. @schferbe, the |
@azizur, the line is from a helper function in case the online server is used. The relevant lines are in https://github.com/adrianvlupu/C4-Builder/blob/master/build.js#L584, https://github.com/adrianvlupu/C4-Builder/blob/master/build.js#L158 and https://github.com/adrianvlupu/C4-Builder/blob/master/build.js#L102 |
I wouldn't necessarily go with relative include paths. If you work on the same project with a couple of colleagues, or on a couple different devices, this forces every environment to stick to a certain folder structure. From my point of view (and maybe it's already possible and I simply missed it), it should be good enough to provide a way to configure PlantUML's include path. |
This is another problem with C4-PlantUML local install. I use it locally by replacing the C4-Plantuml web paths to local paths. Repro:
The problem with this is that when another developer downloads the docs the absolute path must be replaced everywhere. Summary:
works but the site generation
works fine in both VS and site -w but makes it much harder for a team to work on the project until they all keep the same folder structure, a.k.a. same absolute paths. Possible solution: Is it hard to add a path in the config that changes that? |
Hi, any update on this issue? |
Do you accept PRs? I can try to work on this. |
Yes, I accept PRs. Last weekend I managed to merge a couple of them. I would really appreciate a PR. There's a branch I use for local development at https://github.com/adrianvlupu/C4-PlantUML/tree/local, that could be a good starting point. |
I solved this for now while the final solution is being developed using a local plantuml server in a docker container and adding the capability to serve as a static file the core C4 files, and modify C4 files to get the dependencies from this local server. Here is my initial approach: https://github.com/darielrll/c4-builder-example |
This (work on local solution) sounds promising. My current workaround is as such:
I noticed that if I have a file
it works in VS Code. The solution you work on... Could it scan that |
|
thanks , and my additional settings: |
The C4-PlantUML documentation suggests that you can include the
puml
files locally with relative paths. I tried this withc4builder site
and"generateLocalImages": true
but that results in errors.Is there a way that I missed?
The text was updated successfully, but these errors were encountered: