Skip to content
This repository has been archived by the owner on Jul 1, 2021. It is now read-only.

Use require_relative instead of require to speed up requires #175

Merged
merged 1 commit into from
Jul 31, 2020
Merged

Conversation

tas50
Copy link
Contributor

@tas50 tas50 commented Feb 12, 2020

require_relative is faster as it doesn't require traversing the
filesystem looking for the require.

See benchmarks here: rspec/rspec-expectations#476 (comment)

Signed-off-by: Tim Smith [email protected]

require_relative is faster as it doesn't require traversing the
filesystem looking for the require.

See benchmarks here: rspec/rspec-expectations#476 (comment)

Signed-off-by: Tim Smith <[email protected]>
@jrgarcia
Copy link
Contributor

I'd love to merge this, but unfortunately the library currently still supports Ruby 1.8.7. I'm going to leave this open because I plan to drop support for all non-stable Ruby versions. Once I do that, I'll merge this in. I'm not sure when I'll have the time to do it, but I'd love to do that soon.

@Fryguy
Copy link
Contributor

Fryguy commented Mar 10, 2020

Ruby 1.8.7 was dropped in #178

@jrgarcia
Copy link
Contributor

Only the CI support was dropped. Support for 1.8.7 is still in the library itself https://github.com/vmware/rbvmomi/blob/master/rbvmomi.gemspec#L32

@tas50
Copy link
Contributor Author

tas50 commented Jul 31, 2020

@jrgarcia any update on dropping Ruby 1.8.7 support?

@jrgarcia
Copy link
Contributor

I'll plan on doing this today.

@jrgarcia jrgarcia merged commit 439e23e into vmware-archive:master Jul 31, 2020
@tas50
Copy link
Contributor Author

tas50 commented Jul 31, 2020

Thanks @jrgarcia

@jrgarcia
Copy link
Contributor

No problem! Sorry it took so long! I'm changing a couple more things and then I'll push out a new major Gem release.

@jrgarcia
Copy link
Contributor

Thanks so much for this! Just pushed rbvmomi v3 to RubyGems with these changes.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants