Skip to content

Conversation

@runephilosof
Copy link

Fixes: #685

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

Fixes: ruby#685

This feature can easily break how you use other gems like factory_bot or prawn.
@tompng
Copy link
Member

tompng commented Oct 9, 2025

👍

As described in y.rb comment

An alias for Psych.dump_stream meant to be used with IRB.

Removing y and y.rb can be considered just a feature drop from IRB. As a maintainer of IRB, I think dropping it is fine and worth enough.
In addition, similar method j and jj defined in json are already deprecated. Removing y makes sense.

About CI failure: Looks like psych.gemspec needs to be modified too. Need to remove "lib/psych/y.rb" line.

@tenderlove
Copy link
Member

I'm fine with removing the automatic require. But I kind of think we should leave psych/y around. If people really want to use it they could require the file. Note that this feature was just kept around for compatibility with Syck. I'd rather not define top-level functions, but I also want to make sure that if people are depending on that function that there is a way to get it back.

runephilosof added a commit to runephilosof/psych that referenced this pull request Oct 28, 2025
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'
```
@hsbt hsbt closed this Oct 29, 2025
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)

4 participants