yesodSpecWithSiteGenerator with an argument#1485
Conversation
yesod-test/Yesod/Test.hs
Outdated
| , yedResponse = Nothing | ||
| } | ||
|
|
||
| yesodSpecWithSiteGenerator' :: YesodDispatch site |
There was a problem hiding this comment.
What's the purpose of this function? Why would I use this function instead of yesodSpecWithSiteGenerator? We definitely need a Haddock for this. (Also, when written, we need a @since comment, ChangeLog entry, and cabal file version bump.)
|
Let's first see if we can agree that this addition is useful.
The use-case is as follows: I have a I hope the above clears up the need and the purpose of this new function.
Will do that if you agree that this is a useful change. |
|
I wasn't objecting to the inclusion of the function, something which takes an argument like this seems fine. But it needs a more descriptive name and Haddocks. Also, while we're at it: can't |
f4919a0 to
a8f5418
Compare
|
@snoyberg Ready for more review. |
|
Thanks! |
Before submitting your PR, check that you've:
@sincedeclarations to the Haddocks for new, public APIsAfter submitting your PR:
^ I'll complete the checkboxes if you approve of the change.