maxtext icon indicating copy to clipboard operation
maxtext copied to clipboard

Reviewed MaxText README.md

Open mikegre-google opened this issue 10 months ago • 2 comments

I updated the MaxText/README.md file for clarity and for it to follow the devsite style guide (mostly).

mikegre-google avatar Apr 11 '24 21:04 mikegre-google

I addressed the feedback and removed the extra new lines from README.md

mikegre-google avatar Apr 12 '24 18:04 mikegre-google

I squashed my commits and pushed the changes to a new branch. I got a message saying "warning you are about to push to the main branch" and, while I'm pretty sure I can't push to this repo's main branch, I didn't want to take any chances of messing anything up since my forked repo has not been synced with this repo. Please let me know if there is anything else I need to do. Michelle Yoo told me that after I squash my commits, and my changes pass whatever testing is done, a CL will be created. but I don't know if that CL will be assigned to me or if I get notified when that happens. Sorry for the newbie questions. Once I go through this process, I won't have to ask all these questions again. Thanks for your understanding.

mikegre-google avatar Apr 16 '24 22:04 mikegre-google