search

QuantEcon / ContinuousDPs.jl

Continuous state dynamic programming
BSD 3-Clause "New" or "Revised" License
13 stars 10 forks source link

Update for julia 1.4: add {Project,Manifest}.toml, rm require #27

Closed greimel closed 4 years ago

greimel commented 4 years ago

tests pass locally

oyamad commented 4 years ago

@greimel Many thanks!

Just one question, do we need to include Manifest.toml files?

greimel commented 4 years ago

They are not necessary. Opinions differ, see https://discourse.julialang.org/t/does-manifest-toml-belong-in-the-repository/12029

Shall I remove them?

oyamad commented 4 years ago

Yes please, thanks!

greimel commented 4 years ago

done

greimel commented 4 years ago

Please add the email address related to your github account (or remove the stub) to Project.toml before you merge. Thanks!