Skip to content

Version comment in yaml#283

Merged
sammy-da merged 2 commits into
mainfrom
version-comment-in-yaml
Jun 26, 2026
Merged

Version comment in yaml#283
sammy-da merged 2 commits into
mainfrom
version-comment-in-yaml

Conversation

@sammy-da

@sammy-da sammy-da commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

dpm will now include # <semver> comment next to the component / dar URI in the daml.yaml / multi-package.yaml when it running any of the add / install / update commands

cat /var/folders/df/x806g1dd1v3b956tklkp5trc0000gq/T/TestSuiteTestAddingExistingComponentWithFloatyTagBumpsToNewerVe3513982764/001/daml.yaml
components:
  - pre-existing:1.2.3
  - oci://127.0.0.1:49263/newly/added:latest@sha256:88c48bf9679d591aa2625b39d1946c26b265162612a41e8ca50a2f085d6d2af8 # 4.5.6

@sammy-da sammy-da force-pushed the version-comment-in-yaml branch 4 times, most recently from 88723ad to aed0e2c Compare June 26, 2026 17:20
@sammy-da sammy-da force-pushed the version-comment-in-yaml branch from aed0e2c to 049f95b Compare June 26, 2026 17:23
@sammy-da sammy-da marked this pull request as ready for review June 26, 2026 17:29
@dasormeter

Copy link
Copy Markdown
Contributor

if you try to install a new tag for a component that already exists clearer text would be

❯ go run ../cmd/dpm/main.go add component oci://europe-docker.pkg.dev/da-images/public-all/components/damlc:3.6.0-snapshot.20260616.14717.0.v23f80fec
component "oci://europe-docker.pkg.dev/da-images/public-all/components/damlc" is already used in this project and will ...

Pulling...
Successfully installed and added component "oci://europe-docker.pkg.dev/da-images/public-all/components/damlc:3.6.0-snapshot.20260616.14717.0.v23f80fec@sha256:8e407a022b76f0eb18131f4cb797f27074bd1b45f0c8ff764cd9a8be27d287aa" to "/Users/brianhealey/g/dpm/skeleton/multi-package.yaml"

@sammy-da sammy-da force-pushed the version-comment-in-yaml branch from 9a5d1cb to 17c7411 Compare June 26, 2026 17:39
@sammy-da sammy-da force-pushed the version-comment-in-yaml branch from 17c7411 to 71e66ee Compare June 26, 2026 17:40
@sammy-da sammy-da enabled auto-merge (squash) June 26, 2026 18:10
@sammy-da sammy-da merged commit f9f3d45 into main Jun 26, 2026
5 checks passed
@sammy-da sammy-da deleted the version-comment-in-yaml branch June 26, 2026 18:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants