Skip to content

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

Merged
merged 59 commits into from
Dec 22, 2023
Merged

Conversation

LiliamLeme
Copy link
Collaborator

Adding setup information and readme information:

Docs folder

  1. JSON file
  2. Setup for testing libraries
    3)link to this information in the readme

@LiliamLeme
Copy link
Collaborator Author

Adding setup information and readme information:

Docs folder

  1. JSON file
  2. Setup for testing libraries
    3)link to this information in the readme

remove additional libraries.
 no need
Copy link
Collaborator

@arjendev arjendev left a 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 :)

LiliamLeme and others added 11 commits December 15, 2023 12:49
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
another reference forgotten
ydaponte
ydaponte previously approved these changes Dec 21, 2023
Copy link
Collaborator

@ydaponte ydaponte left a 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!

Copy link
Collaborator

@arjendev arjendev left a 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.

LiliamLeme and others added 7 commits December 22, 2023 11:59
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
fix references
@ydaponte ydaponte added the documentation Improvements or additions to documentation label Dec 22, 2023
remove the code
LiliamLeme and others added 6 commits December 22, 2023 12:22
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)
Copy link
Collaborator

@arjendev arjendev left a 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!

@LiliamLeme LiliamLeme requested a review from ydaponte December 22, 2023 13:20
Copy link
Collaborator

@ydaponte ydaponte left a comment

Choose a reason for hiding this comment

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

LGTM! Thank you!

@ydaponte ydaponte added this to the 2023-12 milestone Dec 22, 2023
@LiliamLeme
Copy link
Collaborator Author

mering readme

@LiliamLeme LiliamLeme merged commit de61b82 into microsoft:main Dec 22, 2023
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
Development

Successfully merging this pull request may close these issues.

3 participants