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

Updated input-output.md #399

Closed
wants to merge 2 commits into from
Closed

Conversation

ArshadAriff
Copy link
Contributor

The typos and the errors in the documentation are resolved

The typos and the errors in the documentation are resolved
docs/docs/scheme-intro/input-output.md Show resolved Hide resolved
@@ -345,8 +345,7 @@ Some scheme implementations have `with-input-from-string` procedure:
;; ==> 10 (1 2 3 4)
```

Procedure `with-input-from-string` changes the current-input-port so you can use `read` without an
argument.
The procedure `with-input-from-string` changes the current-input-port so you can use read without an argument.
Copy link
Owner

@jcubic jcubic Oct 30, 2024

Choose a reason for hiding this comment

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

Please put:

an argument

Into separated line. Also, please don't remove the Markdown formatting, you delete code from `read`

@ArshadAriff
Copy link
Contributor Author

Sir i have renewed the PR

@jcubic
Copy link
Owner

jcubic commented Oct 30, 2024

You don't need to close and open new PR, you can commit here and answer the comments I did in the Code Review. This is not how you contribute. Now I need to do a review again on your new PR.

@ArshadAriff
Copy link
Contributor Author

Oh really sorry sir working on it

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.

2 participants