Skip to content

Conversation

@martinflorian-da
Copy link
Contributor

Mainly I want the last commmit there to contain [ci]...

Signed-off-by: Robert Autenrieth <robert.autenrieth@digitalasset.com>
Signed-off-by: Robert Autenrieth <31539813+rautenrieth-da@users.noreply.github.com>
Co-authored-by: Itai Segall <itai.segall@digitalasset.com>
@martinflorian-da martinflorian-da force-pushed the martinflorian-da/cni-2301-final-merge branch from e051ec7 to 847c8ee Compare October 20, 2025 10:32
@martinflorian-da
Copy link
Contributor Author

Mainly I want the last commmit there to contain [ci]...

Ah sadly if I'm doing "merge commit" this also means that we need to do a full [ci] run here 🙄

@moritzkiefer-da if you have a better idea don't let me slow you down; probably we could also just remove the branche protection for canton-3.4 briefly

@moritzkiefer-da
Copy link
Contributor

@moritzkiefer-da if you have a better idea don't let me slow you down; probably we could also just remove the branche protection for canton-3.4 briefly

not sure what you're trying to achieve here?

@martinflorian-da
Copy link
Contributor Author

not sure what you're trying to achieve here?

Get a commit on there that contains [ci] without needing to do a full [ci] on a PR first (that doesn't look like it needs it).

@moritzkiefer-da
Copy link
Contributor

I wouldn't overoptimize for that, just keep the [ci] on the PR.

…2301-final-merge

[force] [force]

Signed-off-by: Martin Florian <martin.florian@digitalasset.com>
@martinflorian-da martinflorian-da force-pushed the martinflorian-da/cni-2301-final-merge branch from a9e489d to 02629e7 Compare October 20, 2025 12:21
@martinflorian-da martinflorian-da merged commit 2dbbab8 into canton-3.4 Oct 20, 2025
5 of 40 checks passed
@martinflorian-da martinflorian-da deleted the martinflorian-da/cni-2301-final-merge branch October 20, 2025 12:24
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.

5 participants