-
Notifications
You must be signed in to change notification settings - Fork 228
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
Conveniences ought to have a home #124
Comments
I like the idea! Especially as a separate framework. |
I love the idea of linking to gists or providing a |
The one problem with a |
I was thinking it'd be a different repo for that reason. |
Agreed, a separate framework & repo seems the best approach. In my experience however it is easy for the two frameworks to get out of sync, e.g. If you forget to update the contrib after a PR in the main repo. Is there a proven process for this? Just my 5 cents for the name: |
Well the "contrib" version would reference a specific That also means that one can simply change their |
Understood. I was hoping there was some sort of safeguard/check that would prevent the contrib library from lagging behind. |
@Thomvis: We could add it as a private dependency and then test it.
👏 I love it 😄 |
Since Result just hit 1.0, I'd expect the API to be pretty stable and at least no backwards incompatible changes anytime soon. So I wouldn't expect this to be a problem. |
Can we provide the contrib stuff in a submodule of the Result module? e.g.: import Result.Oriented
let 🎉 = … If so, we could ship it in this repo, which would be a lot more convenient both for contributors and end users. Presumably Carthage wouldn’t be too fussed since it’d end up being in the same compilation unit, just a separate namespace? (I’m not entirely sure how any of that works!) |
So uh, did this ever turn into something tangible? |
@Evertt: Not thus far. |
Too bad, I really liked the idea. I happen to be a great fan of automagic stuff and since I know it won't be implemented in this library I'd love to know there's a central place where I can find plugins like that. |
@Evertt: Nothing’s been established yet, but that’s not to say that you couldn’t be the one to establish it, if you want. If you’re interested, I’d suggest starting very minimally, so that the contrib stuff’s home can be reviewed independently of any conveniences added therein. |
We’ve had a few PRs along the lines of #123, where someone has submitted a scratch for a not-uncommon itch. We’ve been turning them down—mostly—thus far, but maybe we should instead be finding them a better home.
As I understand it, this is the sort of thing that
git
’scontrib
dir exists to enable. Maybe we should consider something similar. Some ideas:Contrib
(sub?)framework encapsulating conveniences and experimentsThe text was updated successfully, but these errors were encountered: