Skip to content

fix(sync): don't duplicate editable projects in env.toml that have no identifiers, include publisher in lockfile#381

Merged
andrius-puksta-sensmetry merged 1 commit into
sensmetry:mainfrom
andrius-puksta-sensmetry:pr/fix-lock-env
Jun 4, 2026
Merged

fix(sync): don't duplicate editable projects in env.toml that have no identifiers, include publisher in lockfile#381
andrius-puksta-sensmetry merged 1 commit into
sensmetry:mainfrom
andrius-puksta-sensmetry:pr/fix-lock-env

Conversation

@andrius-puksta-sensmetry
Copy link
Copy Markdown
Collaborator

@andrius-puksta-sensmetry andrius-puksta-sensmetry commented Jun 4, 2026

When sysand sync is run multiple times, editable projects present in sysand-lock.toml and not having an identifier will be included each time into .sysand/env.toml, causing duplication. Among such projects is generally the current project that the sync is run on.

Also fixes a bug that caused publisher to be omitted in sysand-lock.toml and by extension some .sysand/env.toml entries.

@andrius-puksta-sensmetry andrius-puksta-sensmetry added the bug Something isn't working label Jun 4, 2026
@consideRatio
Copy link
Copy Markdown
Collaborator

We haven't locked in what we write within the parenthesis like fix(sync) but perhaps individual commands could fit there - so instead of fix: sync: it would be fix(sync):

@andrius-puksta-sensmetry andrius-puksta-sensmetry changed the title fix: sync: don't duplicate editable projects in env.toml that have no identifiers, include publisher in lockfile fix(sync): don't duplicate editable projects in env.toml that have no identifiers, include publisher in lockfile Jun 4, 2026
… identifiers, include publisher in lockfile

Signed-off-by: Andrius Pukšta <andrius.puksta@sensmetry.com>
@andrius-puksta-sensmetry andrius-puksta-sensmetry merged commit 4ae75ba into sensmetry:main Jun 4, 2026
54 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants