Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

deivid-rodriguez
Copy link
Contributor

No description provided.

@olleolleolle
Copy link
Contributor

Is this a way to deal with "double requires gives us warnings"?

@deivid-rodriguez
Copy link
Contributor Author

I'm not creating this PR to address any particular issue. I've just been making this change whenever we vendor a stdlib inside bundler/rubygems. I believe require_relative is better for internal requires, and it also minimizes the changes we need to make in our vendored copy (because the requires can remain unchanged).

@olleolleolle
Copy link
Contributor

@deivid-rodriguez Thanks for adding this context I was missing. Sounds good.

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.

3 participants