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

Resolved the documentation errors #400

Closed
wants to merge 4 commits into from

Conversation

ArshadAriff
Copy link
Contributor

No description provided.

The typos and the errors in the documentation are resolved
@ArshadAriff
Copy link
Contributor Author

I have resolved the errors with the documentation

Copy link
Owner

@jcubic jcubic left a comment

Choose a reason for hiding this comment

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

Please don't close the PR. All commits you made will be added here. PR is dynamic, it's not frozen when you create it.

@@ -159,7 +159,7 @@ If you first evaluated the code that writes into a file `test-2.scm` you should
was written into it. Note that `read` was called two times and read two expressions (two strings).

Same as with output-port you need to call `close-input-port` procedure to close the input port. But
Scheme, same as with output port have handy function that allow to close the port automagically.
Scheme, same as with output port have handy function that allow to close the port automatically.
Copy link
Owner

Choose a reason for hiding this comment

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

You didn't fixed this:

This is not an error, see "automagically" in Jargon File.

@ArshadAriff
Copy link
Contributor Author

Sir i changed all the typos are these enough sir! anything more?

@jcubic
Copy link
Owner

jcubic commented Oct 30, 2024

I'm not gonna marge this, you did change random stuff like syntax of the code, replacing single quote with double quote or replacing * with -. The task was to change grammar of the text, to change the syntax of Markdown and code.

Also, I still, see that you changed automagically to automatically, that I specifically asked not to do.

@ArshadAriff
Copy link
Contributor Author

Sir i really want to contribute please guide me on exactly what to do!

@jcubic
Copy link
Owner

jcubic commented Oct 31, 2024

Remove the changes to the Markdown syntax and source code. And remove the change to replace "automagically" with "automatically".

To phrase it simpler: Only modify English words.

@ArshadAriff ArshadAriff closed this by deleting the head repository Oct 31, 2024
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