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

Fix documentation typo, ininin #842

Merged
merged 1 commit into from
Jan 25, 2024
Merged

Conversation

elebow
Copy link
Contributor

@elebow elebow commented Jan 23, 2024

No description provided.

lib/irb.rb Outdated
@@ -207,7 +207,7 @@
# in file <tt>~/.irb_history</tt>
# (or, if a {configuration file}[rdoc-ref:IRB@Configuration+File]
# is found, in file +.irb_history+
# inin the same directory as that file).
# in the same directory as that file).
Copy link
Member

Choose a reason for hiding this comment

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

In addition to the typo, the entire sentence reads a bit weird 🤔 Would you mind improving it altogether?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Pushed a larger change.

@elebow elebow force-pushed the fix-doc-typo-inin branch from cacfa0e to a1632b2 Compare January 25, 2024 06:00
# By default, \IRB stores a history of up to 1000 input commands in a
# file named <tt>.irb_history</tt>. The history file will be in the same directory
# as the {configuration file}[rdoc-ref:IRB@Configuration+File] if one is found, or
# in <tt>~/</tt> otherwise.
Copy link
Contributor

Choose a reason for hiding this comment

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

I like that it added more clarity on the behaviour, too! 🚀

Copy link
Contributor

Choose a reason for hiding this comment

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

PS: I think this is good to go!

Copy link
Member

@st0012 st0012 left a comment

Choose a reason for hiding this comment

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

This is a great improvement. Thank you very much 👍

@st0012 st0012 merged commit bbabf81 into ruby:master Jan 25, 2024
28 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

3 participants