-
Notifications
You must be signed in to change notification settings - Fork 61
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
Add searchable cradle dependencies #42
Conversation
Extends hie.yaml file format with the "dependencies" object. Extends Bios cradle type with an optional FilePath to obtain cradle dependencies. Set for each cradle type the cradle dependencies. Reorganise parsing of "hie.yaml".
cradleRootDir = wdir | ||
, cradleOptsProg = CradleAction "obelisk" (obeliskAction wdir) | ||
obeliskCradleDependencies :: FilePath -> IO [FilePath] | ||
obeliskCradleDependencies _wdir = return [] |
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.
Couldnt find any obelisk files in a very shallow search (mainly the obelisk gh page)
I think just needs a update to the |
cb1a46a
to
a2a7766
Compare
a2a7766
to
58de5c1
Compare
Surely the |
My reasoning was these are the depedencies of a cradle and a cradle specification consists of a type of cradle and dependencies to that cradle. |
Can you change the Then I will merge it. The |
Extends hie.yaml file format with the "dependencies" object.
Extends Bios cradle type with an optional FilePath to
obtain cradle dependencies from.
Set for each cradle type the cradle dependencies.
Reorganise parsing of "hie.yaml".
hie.yaml
a bit more, smoke testing succeededNote: parsing of hie.yaml is less strict now, if there is a field such as "dependecies" (note missing 'n'), it will parse, assuming the correct field "dependencies" is not defined.
Closes #35
Makes #39 obsolete