Other than a minor style nit, looks good!

Thanks, I have added the style fix.

It would also be good to add a test that checks that the new printing works ok. Just comment if you need help how to do that.

Thanks, I would need a little help on how to write test to check this functionality. If you could point me to some resources it would be great! Thanks again.

What this should fix is the printing of this example:

julia> sprint(show, Base.Dict[], context=(:compact=>false, :module=>nothing))
"Base.Dict[]"

So, what would be good is adding an @test call somewhere such as test/show.jl that makes sure that prints exactly like that (and isn't missing the Base. part)

Marked "forget me not" yet seemingly forgotten... 😭

Hi @vtjnash!
I've tried writing the test you've mentioned. I placed it in test/strings/io.jl since the function sprint is defined in base/strings/io.jl. I hope its alright. Please review the same!

Am I correct in saying that this PR is no longer necessary now that the following two PRs have been merged?

• propagate context through a few more printing functions #46716
• Add test for sprint (fixes: #45066) #48474