Issue #17473 has been updated by hsbt (Hiroshi SHIBATA). Eregon (Benoit Daloze) wrote in #note-26:
That PR is actually easy to review as explained in its description, it restores pathname.rb as it used to be and includes all changes to pathname.rb since then, see https://github.com/ruby/pathname/pull/53/files/3736eab91f783cb087f873b99c25f.... IOW, it only adds code in pathname.rb and removes code in pathname.c, and then has separate commits for all other changes. Making multiple smaller PRs does not make sense, especially since the Ruby code is just the one from before the migration to C.
Please separate the small PRs. I want to reduce the side effect like https://github.com/ruby/ruby/pull/13906. And if you have another issue or proposal, please file new issue. ---------------------------------------- Feature #17473: Make Pathname to embedded class of Ruby https://bugs.ruby-lang.org/issues/17473#change-114099 * Author: hsbt (Hiroshi SHIBATA) * Status: Closed * Assignee: akr (Akira Tanaka) ---------------------------------------- pathname is one of most useful utility class of Ruby. I'm happy to use Pathname without require it. Any thought? -- https://bugs.ruby-lang.org/