-
Notifications
You must be signed in to change notification settings - Fork 37
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
Readme: a better positioning with ML loggers #71
Conversation
Should Jorge review for style? |
@jorgeorpinel I'd appreciate it if you can take a look. |
@dmpetrov @dberenbaum I suggest that we merge this and we can do the style review asynchronously after (even editing file directly). This is early stage, not the final pass on this README, it's fine to move faster in this case I think. WDYT? |
Sure, no need to hold things up. |
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.
Just copy edits
Co-authored-by: Jorge Orpinel <[email protected]>
Co-authored-by: Jorge Orpinel <[email protected]>
Co-authored-by: Jorge Orpinel <[email protected]>
Co-authored-by: Jorge Orpinel <[email protected]>
Co-authored-by: Jorge Orpinel <[email protected]>
Co-authored-by: Jorge Orpinel <[email protected]>
Co-authored-by: Jorge Orpinel <[email protected]>
Co-authored-by: Jorge Orpinel <[email protected]>
thank you @jorgeorpinel ! I applied all the suggestions and merged it. |
No description provided.