Skip to content

[ci] move all npm packages to @canton-network#5750

Merged
isegall-da merged 8 commits into
mainfrom
isegall/npm-ns
Jun 3, 2026
Merged

[ci] move all npm packages to @canton-network#5750
isegall-da merged 8 commits into
mainfrom
isegall/npm-ns

Conversation

@isegall-da
Copy link
Copy Markdown
Contributor

Part of #5614 (the splice half)

Pull Request Checklist

Cluster Testing

  • If a cluster test is required, comment /cluster_test on this PR to request it, and ping someone with access to the DA-internal system to approve it.
  • If a hard-migration test is required (from the latest release), comment /hdm_test on this PR to request it, and ping someone with access to the DA-internal system to approve it.
  • If a logical synchronizer upgrade test is required (from canton-3.5), comment /lsu_test on this PR to request it, and ping someone with access to the DA-internal system to approve it.

PR Guidelines

  • Include any change that might be observable by our partners or affect their deployment in the release notes.
  • Specify fixed issues with Fixes #n, and mention issues worked on using #n
  • Include a screenshot for frontend-related PRs - see README or use your favorite screenshot tool

Merge Guidelines

  • Make the git commit message look sensible when squash-merging on GitHub (most likely: just copy your PR description).

Signed-off-by: Itai Segall <itai.segall@digitalasset.com>
Signed-off-by: Itai Segall <itai.segall@digitalasset.com>
import { runSvCantonForAllMigrations } from '@lfdecentralizedtrust/splice-pulumi-sv-canton/pulumi';
import { runSvProjectForAllSvs } from '@lfdecentralizedtrust/splice-pulumi-sv/pulumi';
import { runSvCantonForAllMigrations } from '@canton-network/splice-pulumi-sv-canton/pulumi';
import { runSvProjectForAllSvs } from '@canton-network/splice-pulumi-sv/pulumi';
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These don't have matching changes in sv/package.json or sv-canton/package.json. (There's a lot of @lfdecentralizedtrust/ left too, not sure how incremental you want to be.)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Haa... No, I definitely meant to be complete, not sure how I messed up the global search

Signed-off-by: Itai Segall <itai.segall@digitalasset.com>
Signed-off-by: Itai Segall <itai.segall@digitalasset.com>
Signed-off-by: Itai Segall <itai.segall@digitalasset.com>
Signed-off-by: Itai Segall <itai.segall@digitalasset.com>
Signed-off-by: Itai Segall <itai.segall@digitalasset.com>
Signed-off-by: Itai Segall <itai.segall@digitalasset.com>
@isegall-da
Copy link
Copy Markdown
Contributor Author

@stephencompall-DA I think this is good to go now. Care to review again please?

Copy link
Copy Markdown
Contributor

@stephencompall-DA stephencompall-DA left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, confirmed that this is comprehensive too

@isegall-da isegall-da merged commit 9f5a147 into main Jun 3, 2026
66 checks passed
@isegall-da isegall-da deleted the isegall/npm-ns branch June 3, 2026 20:08
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