-
Notifications
You must be signed in to change notification settings - Fork 34
Adding additional setup information #45
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
Conversation
Adding the images to the setup file
JSON Files step by step. Adding images
adding the ```python
adding the links to the setup docs and working on the readme
Adding setup information and readme information: Docs folder
|
remove additional libraries. no need
removed synapse
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @LiliamLeme !!
Sharing some initial comments, will do another review once they have been addressed :)
removing manual copy of fabric json adding the git integration docs
fixing poetry reference
coma missing
removing #
add more information isntead of suggestion
updating the PR feedbacks
small typo
another reference forgotten
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@LiliamLeme - I'm approving not to block the PR - but please address the image comment before merging. Other than that LGTM. Thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd like to review the docs before going to main, so will mark the PR as "request changes" for now.
adding image from image folder
adding images from the folder
Co-authored-by: Yennifer Santos <46519584+ydaponte@users.noreply.github.com>
Co-authored-by: Yennifer Santos <46519584+ydaponte@users.noreply.github.com>
Co-authored-by: Yennifer Santos <46519584+ydaponte@users.noreply.github.com>
Co-authored-by: Yennifer Santos <46519584+ydaponte@users.noreply.github.com>
no capital letter
non capital letter
non capital letters
non capital letters
fix references
remove the code
fixing the number that were not in the right order adding link to the folder example reference instead of listing the code
[examples](/examples)
typos and number sequence
adjusting the link for examples -> [examples](/examples)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great Liliam, thank you for your contribution and for being so patient with me!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Thank you!
mering readme |
Adding setup information and readme information:
Docs folder
3)link to this information in the readme