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

Use defaultConfigWithSandbox so that we pickup $HASKELL_PACKAGE_SANDBOX.... #8

Merged
merged 1 commit into from
Jun 21, 2014

Conversation

smobs
Copy link
Contributor

@smobs smobs commented Jun 18, 2014

... This fixes issue #7
defaultConfigWithSandbox returns an IO Config so I moved the call to a separate line.
Data.Default is no longer used so I removed it.
I've only tested it on my one OSX machine, if you have concerns I can do more testing.
This is the first time I've created a pull request so please excuse any rookie mistakes.

@bergmark bergmark merged commit 2bf51e9 into faylang:master Jun 21, 2014
bergmark added a commit that referenced this pull request Jun 21, 2014
@bergmark
Copy link
Member

Woops forgot to look at this, sorry.

Looks great, thanks!

I removed the data-default dependency from the cabal file as well since it wasn't used anymore.

I released this as snaplet-fay 0.3.3.7

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