Skip to content

Allow setting rcfile path via flag#49

Merged
codic12 merged 1 commit intocodic12:mainfrom
lambdadog:lambdadog/add-config-opt
Apr 7, 2022
Merged

Allow setting rcfile path via flag#49
codic12 merged 1 commit intocodic12:mainfrom
lambdadog:lambdadog/add-config-opt

Conversation

@lambdadog
Copy link
Contributor

Also moves rcfile loading to worm.nim rather than events.nim, which feels more sensible.

I kept this simple and used std/parseopt because I didn't really want to pull in a dependency for command-line parsing since you weren't using one in wormc, but if using something more robust would be preferred I can do that instead (this is also why I only added a long opt and not a short one, since parseopt usage is quite verbose).

Also moves rcfile loading to worm.nim rather than events.nim, which
feels more sensible.
@codic12
Copy link
Owner

codic12 commented Apr 7, 2022

Looks good to me, thanks for all the PRs!

@codic12 codic12 merged commit 725376c into codic12:main Apr 7, 2022
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.

2 participants