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

Conveniences ought to have a home #124

Open
robrix opened this issue Nov 29, 2015 · 14 comments
Open

Conveniences ought to have a home #124

robrix opened this issue Nov 29, 2015 · 14 comments

Comments

@robrix
Copy link
Contributor

robrix commented Nov 29, 2015

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’s contrib dir exists to enable. Maybe we should consider something similar. Some ideas:

  • a Contrib (sub?)framework encapsulating conveniences and experiments
  • a section of the readme linking to gists with these things
  • just including them in the framework and being done with it
@NachoSoto
Copy link
Contributor

I like the idea! Especially as a separate framework.

@gfontenot
Copy link
Member

I love the idea of linking to gists or providing a Contrib framework alongside Result. I don't think we should be including them in the main framework though.

@neilpa
Copy link
Member

neilpa commented Nov 30, 2015

The one problem with a Contrib style framework in the same repo is it doesn't play all that nice with carthage.

@NachoSoto
Copy link
Contributor

I was thinking it'd be a different repo for that reason.

@Thomvis
Copy link
Member

Thomvis commented Nov 30, 2015

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: ResultOriented.framework

@NachoSoto
Copy link
Contributor

If you forget to update the contrib after a PR in the main repo. Is there a proven process for this?

Well the "contrib" version would reference a specific Result version so compatibility would always be explicit.

That also means that one can simply change their Carthage reference from Result to this other framework, and their existing usages of Result would continue to work.

@Thomvis
Copy link
Member

Thomvis commented Nov 30, 2015

Well the "contrib" version would reference a specific Result version so compatibility would always be explicit.

Understood. I was hoping there was some sort of safeguard/check that would prevent the contrib library from lagging behind.

@robrix
Copy link
Contributor Author

robrix commented Nov 30, 2015

@Thomvis: We could add it as a private dependency and then test it.

Just my 5 cents for the name: ResultOriented.framework

👏 I love it 😄

@kylef
Copy link
Member

kylef commented Nov 30, 2015

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.

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.

@robrix
Copy link
Contributor Author

robrix commented Dec 28, 2015

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!)

@Evertt
Copy link

Evertt commented Nov 8, 2016

So uh, did this ever turn into something tangible?

@robrix
Copy link
Contributor Author

robrix commented Nov 8, 2016

@Evertt: Not thus far.

@Evertt
Copy link

Evertt commented Nov 8, 2016

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.

@robrix
Copy link
Contributor Author

robrix commented Nov 12, 2016

@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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

7 participants