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 Mermaid dependency and change READMEs #29

Merged
merged 5 commits into from
Feb 27, 2024
Merged

Add Mermaid dependency and change READMEs #29

merged 5 commits into from
Feb 27, 2024

Conversation

Murias10
Copy link
Contributor

Move members to correct place and upgrade deployment part.
Solve dependency problem to generate documentation with diagram tool Mermaid and add it to the docs/README.md

@Murias10 Murias10 added the documentation Improvements or additions to documentation label Feb 26, 2024
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@Murias10
Copy link
Contributor Author

I have checked and the dependency problem I think is now fixed, check it in any case

@Murias10 Murias10 assigned Murias10 and unassigned Murias10 Feb 26, 2024
@Raulms29
Copy link
Contributor

Have you tested it on a computer that doesn't have Mermaid installed? I have it on my computer already so I cannot try.
We could also add, just in case it doesn't work properly, the commands to do the installation manually.

@Murias10
Copy link
Contributor Author

Yes, I deleted the entire project and cloned it from scratch to delete all the modules that I had installed, and it worked correctly

@Murias10 Murias10 self-assigned this Feb 27, 2024
@Raulms29 Raulms29 merged commit a06c651 into master Feb 27, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants