Skip to content
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

Fixed #3124: Added documentation for printf in MemoryIO #3125

Conversation

abhijitiitr
Copy link

Adding the documentation for printf in the MemoryIO class to fix this issue.

@abhijitiitr abhijitiitr force-pushed the documentation/printf_memory_io branch from 5d49aec to fcaf811 Compare August 9, 2016 11:55
@@ -347,6 +347,15 @@ module IO
nil
end

# Writes the formatted string to STDOUT. The format can include
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No, to the given IO.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jhass Correct. Shall correct it and push it again.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can just force push to an existing branch to update a pull request ;)

@abhijitiitr abhijitiitr closed this Aug 9, 2016
@abhijitiitr abhijitiitr deleted the documentation/printf_memory_io branch August 9, 2016 13:07
abhijitiitr added a commit to abhijitiitr/crystal that referenced this pull request Aug 9, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

#printf docs missing
2 participants