- Notifications
You must be signed in to change notification settings - Fork 11
IOGX Integration #33
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
IOGX Integration #33
Conversation
| @zeme-wana Thanks for the PR. I must admit I am slightly reluctant to merge this PR as it seems very specific to IOG's environment. While quickcheck-dynamic has been developed in the context of IOG-funded projects and is currently used mostly within those projects, the "vision" has always been to consider it an open-source project that should appeal to a broader audience. Let's imagine that tomorrow IOG infrastructure disappears over the night (certainly not something I wish to happen), what would be the consequences for this repository? Would the changes in this PR become irrelevant, or even break? Another question: What does this PR brings to casual haskellers using nix but not interested in relying on IOG's infrastructure? |
| @abailly-iohk Thank you for your questions. This PR is very specific to IOG in that it introduces dependences on other IOG repositories, namely (haskell.nix + CHaP + Hackage) and iohk-nix. If (the entirety of) IOG infrastructure were to disappear overnight, the nix code in this repository would break, including CI and the ability to enter a devShell. To the casual haskeller using nix, this PR brings a devShell that comes with a complete Haskell development environment, and the ability to easily extend that shell, including the option to change or add new ghc versions, as well as build against a ghc matrix in CI. The idea of IOGX is to establish a standard way to structure nix code whilst getting most of the common stuff for free (ghc matrices, haskell toolchain, formatters, hydra jobsets, readthedocs site, and more to come) but under the assumption that the repository is built on top of haskell.nix. There is a price and risk to pay for this dependency. And ultimately it's up to you as the maintainer of this repository to decide if IOGX brings enough of an improvement to justify that cost. |
| @zeme-wana Thanks a lot for your detailed answer. I am just a maintainer, not the only one. I would be interested in what other contributors think (ping @UlfNorell @MaximilianAlgehed) |
| While I appreciate the effort, I'm unfortunately inclined to agree with the point that @abailly-iohk is making here. Is there some middle ground we can strike that makes things easy for people working with IOG stuff while not creating a maintenance burden. Finally, I think the only IOG dependency in this repo is |
| I understand your concerns. Let me to clear a few points:
I do appreciate that If the benefits don't win you over then I won't push this further and I will close this PR. |
| I'm willing to buy that argument. So long as things keep working without nix - which they should do. @abailly-iohk what are your thoughts? |
223d12b to 40e0d77 Compare 0ca91b8 to 97522f4 Compare
nix developnow drops you in a shell with a complete Haskell toolchain