Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Switch to using require_relative for local files #154

Merged
merged 1 commit into from
Aug 11, 2024

Conversation

nirebu
Copy link
Contributor

@nirebu nirebu commented Aug 11, 2024

Closes #151

As it is faster than require.

I didn't benchmark this though.

Copy link
Member

@postmodern postmodern left a comment

Choose a reason for hiding this comment

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

Noticed a few instances of Find/Replace changing example code.

lib/ronin/recon/importer.rb Outdated Show resolved Hide resolved
lib/ronin/recon/worker.rb Outdated Show resolved Hide resolved
lib/ronin/recon/worker.rb Outdated Show resolved Hide resolved
@nirebu nirebu force-pushed the nirebu/require-relative branch from 43b9130 to 9da2cc9 Compare August 11, 2024 10:01
@nirebu
Copy link
Contributor Author

nirebu commented Aug 11, 2024

Noticed a few instances of Find/Replace changing example code.

Sorry for not catching those ones! They are now fixed.

@nirebu nirebu requested a review from postmodern August 11, 2024 10:02
@postmodern postmodern merged commit e27d7b1 into ronin-rb:main Aug 11, 2024
4 checks passed
@postmodern postmodern added the enhancement New feature or request label Aug 11, 2024
@postmodern postmodern added this to the 0.1.1 milestone Aug 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Switch to using require_relative for local files
2 participants