Open
Description
yeah we should add JuliaFormatter to this. It would be good to merge as many PRs as possible before because it will create conflicts all over
Originally posted by @matbesancon in #1283 (comment)
NB- the following workflow should only have a minimal amount of pain involved in each remaining open PR, assuming format(".")
on master is committed as a single commit tagged format_commit
:
git checkout feature_branch
git merge format_commit^
(no conflicts, or actual content conflicts that must be resolved should be handled here)julia -e "using JuliaFormatter; format("."); git commit -am "PR format"
git merge format_commit; git checkout --ours .; git commit -a
(we know the only diff is the formatting, so it's safe to blanket-accept the local branch side)git merge master
(should apply cleanly now)
Of course, PRs that would be straightforward to merge should probably be merged first. Here's a (not necessarily 100% correct nor complete) list of the ones that seem like they mostly need a "click merge":
- Update aliastable.jl #1074
- PR: Visualizations of distributions in documentation #1115
- Update laplace.jl #1131
- Add gradlogpdf to Uniform #1133
- return -Inf from logdiffcdf if x == y #1180
- Update skewnormal.jl #1212
- Von mises range fix #1280
- fix some bugs related to the Chernoff distribution. #1283
- Add
minimum
,maximum
,extrema
forAbstractMvNormal
andProduct
#1319 - Implement std(::MultivariateDistribution) #1352
Metadata
Metadata
Assignees
Labels
No labels