-
Notifications
You must be signed in to change notification settings - Fork 15k
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
refactor: replace use of deprecated base::JSONWriter::WriteJson()
#41224
refactor: replace use of deprecated base::JSONWriter::WriteJson()
#41224
Conversation
refactor: use base::WriteJson() in DictionaryValueToNSDictionary() Co-authored-by: Charles Kerr <charles@charleskerr.com>
Co-authored-by: Charles Kerr <charles@charleskerr.com>
Co-authored-by: Charles Kerr <charles@charleskerr.com>
Co-authored-by: Charles Kerr <charles@charleskerr.com>
767044e
to
22339a1
Compare
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.
trop got this one wrong. I'll rebase it.
83a73cc
to
22339a1
Compare
This reverts commit cb6cfdf.
Looks like these are coming back during a 3-way merge for some reason 🤔 |
Apparently the bots are doing the right thing? Looks like #41178 wasn't rebased on 28 for #41195, so those four commits are now upstream and the bots are correct to remove the patches. Thanks to @codebytere for spotting this; I didn't see it 😸 |
No Release Notes |
Backport of #41215
See that PR for details.
Notes: none