-
Notifications
You must be signed in to change notification settings - Fork 368
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
Absolute templates path #1691
Absolute templates path #1691
Conversation
this will mean that files can be build from another directory whilst still finding the package's templates
0953cb0
to
90b3268
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great, just a minor comment about version numbers
yesod/Yesod/Default/Util.hs
Outdated
@@ -64,6 +66,11 @@ addStaticContentExternal minify hash staticDir toRoute ext' _ content = do | |||
globFile :: String -> String -> FilePath | |||
globFile kind x = "templates/" ++ x ++ "." ++ kind | |||
|
|||
-- | `globFile` but returned path is absolute and within the package the Q Exp is evaluated | |||
-- @since 1.6.0.3 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since this introduces a new identifier, the version number should be 1.6.1.0
90b3268
to
8585893
Compare
sorted it :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
Before submitting your PR, check that you've:
@since
declarations to the Haddocks for new, public APIsAfter submitting your PR:
fixes #1690
i'm 99% certain this can't break any usages of
widgetFileReload
orwidgetFileNoReload
because those thoseQ Exp
must have already been evaluated in the correct project. unless someone was jumping through a lot of hoops to run ghc manually