Skip to content

Conversation

@runephilosof
Copy link

Fixes: #685

This feature can easily break how you use other gems like factory_bot or prawn.

#747 (comment)

But I kind of think we should leave psych/y around. If people really want to use it they could require the file.

If you miss the function in Kernel, you can require it interactively or add it to .irbrc:

require 'psych/y'

Fixes: ruby#685

This feature can easily break how you use other gems like factory_bot or prawn.

ruby#747 (comment)
> But I kind of think we should leave `psych/y` around. If people really want to use it they could require the file.

If you miss the function in Kernel, you can require it interactively or add it to `.irbrc`:
```ruby
require 'psych/y'
```
Copy link
Author

@runephilosof runephilosof left a comment

Choose a reason for hiding this comment

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

I guess it could be used in exotic scripts, even though it is IRB. So should this be a major version change?

@runephilosof-abtion
Copy link

@tenderlove Here is the version of the PR that you requested in #747.

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.

rails console on reload: undefined method `end_line=' for nil (NoMethodError)

2 participants