Skip to content

Conversation

mohantym
Copy link
Contributor

@mohantym mohantym commented Oct 1, 2022

Fixed some Grammar mistakes in source.md, source_windows.md and pip.md which were found using inspect code feature of Pycharm.

Fixed few grammer mistakes in source.md
Fixed some Grammar mistakes in source_windows.md - found with Inspect code feature from Pycharm
Fixed typo in pip.md found using inspect code feature of pycharm
@mohantym mohantym changed the title Fixed some Grammar mistakes in source.md and source_windows.md Fixed some Grammar mistakes in install directory Oct 1, 2022
8bitmp3
8bitmp3 previously approved these changes Oct 3, 2022
Copy link
Contributor

@8bitmp3 8bitmp3 left a comment

Choose a reason for hiding this comment

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

Thanks for some of the fixes, appreciate it. Please review software-suggested changes and verify them before submitting them next time, as not all changes may make sense.

@8bitmp3 8bitmp3 self-assigned this Oct 3, 2022
@8bitmp3 8bitmp3 added ready to pull Start merge process lgtm Community-added approval labels Oct 3, 2022
@mohantym
Copy link
Contributor Author

mohantym commented Oct 3, 2022

@8bitmp3 !
Thanks for the review. I will test the build internally through bazel before raising further PR's ( same suggested by @gbaned today)
Thank you!

MarkDaoust and others added 2 commits October 4, 2022 06:13
Co-authored-by: 8bitmp3 <19637339+8bitmp3@users.noreply.github.com>
@copybara-service copybara-service bot merged commit 057d9b0 into tensorflow:master Oct 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm Community-added approval ready to pull Start merge process
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants