Adding initial github actions #6
Conversation
dgkf-roche
changed the title
|
Awesome. Thanks a lot @dgkf-roche ! |
| represents ``h(t)`` | ||
| needs to be implemented for any `struct` that subtypes HazardBasedDistribution | ||
| """ | ||
| function hazard(dist::HazardBasedDistribution, t::Real) end | ||
|
|
||
| @doc raw""" |
There was a problem hiding this comment.
Why removing the @doc macro? It makes, it more explicit where the docstrings are.
Why remove the @doc raw? This is made for docstrings with latex.
See the syntax guide from julia: https://docs.julialang.org/en/v1/manual/documentation/#Syntax-Guide
There was a problem hiding this comment.
Ah yeah, we can add it back in for the latex heavy ones. I just did a find and replace, but it does make sense to use for the ones that need a lot of characters escaped.
| @@ -1,5 +1,4 @@ | |||
| [deps] | |||
| Distributions = "31c24e10-a181-5473-b8eb-7969acd0382f" | |||
| Documenter = "e30172f5-a6a5-5a46-863b-614d45cd2de4" | |||
| JointModels = "54f05ff3-c196-4f1a-acfd-0c2b8e4405db" | |||
There was a problem hiding this comment.
Do the docs render without implicitly loading the JointModels module?
There was a problem hiding this comment.
I don't think so. I don't remember exactly what the issue was, but I think it had some circular dependence. If I remember correctly, it was building the module as part of the documentation process so it can't be a dependency for itself.
When in doubt, I just looked at other projects in the Julia ecosystem to see what they did.
Adds
Testworkflow to build, test and calculate coverage for the moduleDeployworkflow for building a ghpages siteThis PR does not attempt to resolve all package issues, only to get the infrastructure set up as a starting point. It appears that at the very least we have to update the development manifest to test against the latest julia release, or better yet remove the manifest since I think that's only considered a best practice for things like reproducible analysis.