Issue/modify inmanta install guide #2869
Conversation
|
|
||
| OR | ||
|
|
||
| 2. Get the source from `our github page <https://github.com/inmanta/inmanta-dashboard/releases>`_ on the right and under `Assets`, you will see the source. Next, build and install the dashboard. For this you need to have yarn and grunt: |
There was a problem hiding this comment.
I would remove this part, using the pre-built npm is easier
There was a problem hiding this comment.
How should I go about it? keep it or remove it?
Co-authored-by: Wouter De Borger <[email protected]>
Co-authored-by: Wouter De Borger <[email protected]>
|
I would like arnaud to have a look at this as well, he has cleaned up these instructions before and has a much better eye for this than me |
Co-authored-by: arnaudsjs <[email protected]>
Co-authored-by: arnaudsjs <[email protected]>
docs/Makefile
Outdated
| @@ -56,7 +56,6 @@ clean: | |||
| -rm -rf $(PNGs) | |||
|
|
|||
| html: openapi latexpdf | |||
| cp $(BUILDDIR)/latex/inmanta.pdf $(SOURCEDIR) | |||
There was a problem hiding this comment.
I think this has to stay, even if I'm not sure why?
@arnaudsjs do you know why this is?
There was a problem hiding this comment.
Yes. It has to stay. A PDF version of the documentation is included in the HTML version of the documentation. This command moves the PDF from the output directory of the PDF build to the source directory of the HTML build.
|
@Pouriya-Inmanta make sure you use merge-tool and not merge it yourself |
|
Processing this pull request |
# Description * Revised Inmanta install guide for Debian
|
Failed to merge changes into iso3 due to merge conflict.Please open a pull request for these branches separately. |
|
Merged into branches master in da183d1 |
|
Not closing this pull request due to previously commented issues for some of the destination branches. Please open a separate pull request for those branches. You can safely close this pull request and delete the source branch. |
# Description * Revised Inmanta install guide for Debian
|
Processing #2889. |
Pull request opened by the merge tool on behalf of #2869
|
Merged in #2891 after resolving the conflict. |
Description