Split options module into options and optionsutils#9160
Split options module into options and optionsutils#9160PMunch wants to merge 3 commits intonim-lang:develfrom
Conversation
Can you point me to where exactly this was discussed? |
|
Hmm, seems like the discussion on GitHub around the topic was limited to this comment. Must've been primarily on IRC then, but looking through the logs to find out where could take a while. |
|
That comment doesn't seem to mention anything about putting this stuff in a separate module. |
|
Oh sorry, I misunderstood. That was Araqs suggestion, again back on IRC |
|
There is a better way to do this, stay tuned... |
This is mostly work by @PMunch done in nim-lang#9160.
|
Since PR #10797 (which was basically a continuation of this one) was rejected, can/should we also close this one? Also, we should decide what to do with the two PRs following this one:
|
|
I've since created a package that implements all these things, which should be available in Nimble. So I'm fine with closing all the PRs for now. |
|
Thanks @PMunch, closing this and the related PRs. |
As discussed under #8358 to add onything fancy to options it should be split into options and optionsutils. This PR is a dependency for future ones that adds functionality to this new module.