-
Notifications
You must be signed in to change notification settings - Fork 117
Update TotW #120 to reflect the OSS availability of absl::Cleanup and absl::Status #501
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
Conversation
…up and absl::Status
|
Friendly ping @derekmauro :) |
|
Thanks for the reminder @CJ-Johnson. I'm actually in the process of getting a new technical writing team up to speed to manage this repo. Please be patient. |
|
Fantastic! Thank you so much :) @derekmauro |
|
Hey @derekmauro ! Just checking in. Were you able to form a team to handle reviews? Thanks! |
|
@axgillies - Can you look into importing this change? Or maybe the internal version is now releasable? I think the context here is that we published the tip before |
|
Thanks for handing it off @derekmauro ! @axgillies Feel free to let me know if you'd like any updates before landing. |
|
Hey @derekmauro It seems this hasn't been picked up yet. Are you able to ping the new reviewer? Thanks! |
|
Thanks for bumping this up, CJ -- I've put this in my queue for this week. |
|
It looks like we have two versions of this Tip that have gone out of sync, one internal and one externally proposed here. The internal version has been updated to add several of the changes in this PR. I'm going to create a new pull request from the internal version. CJ, can you sync to that version and then take another look at your change? |
|
I synced the version of the tip in the repo with the internal version here: CJ, can you review and then update this change? |
I noticed that these two utilities were not being used in the code examples of TotW #120. Hopefully this all checks out!
Also, while I'm here, I fixed the incorrect mailto address from @gmail.com to @google.com