Skip to content

Conversation

@jcalarcon98
Copy link
Contributor

Description

All the README documentation was updated to support the new way of running the project, in this case, everything was run with docker-compose, also documentation was added for the correct use of the "Time Tracker CLI", in charge of generating data fake in the cosmos emulator.

Copy link
Contributor

@bytesantiago bytesantiago left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It seems very good for me. Everything seems to be pretty clear and understandable. Great Job @jcalarcon98 !

Copy link
Contributor

@edgardavid2015 edgardavid2015 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Excellent work Jean Carlos, I just saw two mistakes in the README files.

Copy link
Collaborator

@JosueOb JosueOb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I didn't have any complications following the instructions to set up the backend development environment using Docker. Also, I tested the time tracker CLI and was able to create new entries using the interactive and command-based way.

@jcalarcon98 jcalarcon98 force-pushed the TT-301-update-readme-with-cosmos-emulator-configuration branch from dc4d273 to 8090d33 Compare August 4, 2021 18:36
@sonarqubecloud
Copy link

sonarqubecloud bot commented Aug 4, 2021

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@jcalarcon98 jcalarcon98 merged commit 58dbc15 into master Aug 6, 2021
@jcalarcon98 jcalarcon98 deleted the TT-301-update-readme-with-cosmos-emulator-configuration branch August 6, 2021 02:23
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.

6 participants