digital-asset / daml

The Daml smart contract language
https://www.digitalasset.com/developers
797 stars 199 forks source link

Feature request: daml-config.yaml config to dogfood latest snapshot and LF version #9077

Open shaul-da opened 3 years ago

shaul-da commented 3 years ago

Currently daml assistant notifies only on new releases. I want to configure daml assistant to always use the bleeding edge

  1. Notify me on new snapshot releases
  2. Default to daml install latest --snapshots yes
  3. Configure new projects to latest LF version

I expect if we have this we'll have more people using this configuration and dogfooding the latest and greatest

cocreature commented 3 years ago

I’m a bit sceptical this is a good idea. Even for simple projects or toys, I frequently find myself coming back to them after a few months or weeks and I absolutely hate situations where I cannot build my project anymore with no good way of figuring out which version it did build against in the past

I get the argument that it might get more people to test the latest snapshot and I agree with that but I’m not sure it outweighs the downsides.

Maybe a middleground would be a command that updates your daml.yaml to use the latest snapshot automatically so it becomes less painful to test it but it’s still fixed after that (until you run the command again)?

shaul-da commented 3 years ago

IIUC you're pushing back on (3) and not arguing with (1) and (2). I fully agree that once a project is created it should be fixed, it would be pretty horrible otherwise. But I would like

  1. a new project to default to the latest snapshot and LF as of the time of creation
  2. an easy command to update
cocreature commented 3 years ago

Oh I see, sorry I misread your request. I’m not even opposed to 3. I’m opposed to daml.yaml having a sdk-version: latest-snapshot field which does not lock down the version.

shaul-da commented 3 years ago

Yep, I would never use that