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

Slight change in README #33015

Closed
wants to merge 2 commits into from

Conversation

@aryan-harsh
Copy link

commented Oct 3, 2019

Added slight change to make README more understandable for non-ML folks.

Added slight change to make README more understandable for non-ML folks.
@tensorflow-bot tensorflow-bot bot added the size:XS label Oct 3, 2019
@googlebot

This comment has been minimized.

Copy link

commented Oct 3, 2019

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed (or fixed any issues), please reply here with @googlebot I signed it! and we'll verify it.


What to do if you already signed the CLA

Individual signers
Corporate signers

ℹ️ Googlers: Go here for more info.

@googlebot googlebot added the cla: no label Oct 3, 2019
@aryan-harsh

This comment has been minimized.

Copy link
Author

commented Oct 3, 2019

@googlebot I signed it!

@googlebot

This comment has been minimized.

Copy link

commented Oct 3, 2019

CLAs look good, thanks!

ℹ️ Googlers: Go here for more info.

@googlebot googlebot added cla: yes and removed cla: no labels Oct 3, 2019
@gbaned gbaned self-assigned this Oct 3, 2019
@gbaned gbaned added this to Assigned Reviewer in PR Queue via automation Oct 3, 2019
@gbaned gbaned requested a review from lamberta Oct 3, 2019
PR Queue automation moved this from Assigned Reviewer to Approved by Reviewer Oct 4, 2019
Copy link
Member

left a comment

Thanks. I'm fine with this since it's the intro and we use "ML" in the sentence that follows.

@gbaned

This comment has been minimized.

Copy link

commented Oct 14, 2019

We will not be encouraging one liner grammatical changes as this is expensive process, thank you for your interest.
CC @mihaimaruseac @chanshah

@gbaned gbaned closed this Oct 14, 2019
PR Queue automation moved this from Approved by Reviewer to Closed/Rejected Oct 14, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
PR Queue
  
Closed/Rejected
6 participants
You can’t perform that action at this time.