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

Improve irbrc loading order #808

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

ktopolski
Copy link

Ensures that "project level" .irbrc will always take precedence over "user level" .irbrc.

Addresses part of #674

test/irb/test_init.rb Outdated Show resolved Hide resolved
test/irb/test_init.rb Outdated Show resolved Hide resolved
@st0012 st0012 added the bug Something isn't working label Dec 10, 2023
@st0012
Copy link
Member

st0012 commented Dec 11, 2023

Thanks for the PR. I just tested it manually and it seems to stop loading user-level config completely when project-level config is present? Is it possible to make it also load the user-level config AFTER the project-level config is loaded? Otherwise, it'd be a big breaking change for some users.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Development

Successfully merging this pull request may close these issues.

2 participants