Remove Reference to Kernel.
#9549
Conversation
src/io.cr
Outdated
| @@ -259,7 +259,7 @@ abstract class IO | |||
| end | |||
|
|
|||
| # Writes a formatted string to this IO. | |||
| # For details on the format string, see `Kernel::sprintf`. | |||
| # For details on the format string, see top-level `sprintf`. | |||
There was a problem hiding this comment.
| # For details on the format string, see top-level `sprintf`. | |
| # For details on the format string, see top-level `printf`. |
There was a problem hiding this comment.
thanks! fixed in 8ece9e1
There was a problem hiding this comment.
what's the policy/preference on squashing commits to clean up PRs?
There was a problem hiding this comment.
That doesn't work, that will point to printf in IO. You need to use ::printf
There was a problem hiding this comment.
The policy is you don't care about how many commits are in here, we always squash PRs before merging.
There was a problem hiding this comment.
thanks! fixed in 3ceb8ff
There was a problem hiding this comment.
it doesn't generate a link to the top-level doc page, but it does prevent a link internal to the class
There was a problem hiding this comment.
Oh, maybe it's not implemented in the doc generator... I guess leaving it like that is good for now.
jhass
added
kind:bug
Removes a reference to the
Kernelmodule, which does not exist in Crystal.