Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor(camunda-client): replace zeebe-client by camunda-client #3826

Conversation

mathias-vandaele
Copy link
Collaborator

@mathias-vandaele mathias-vandaele commented Jan 2, 2025

Description

Replace zeebe-client dependency by camunda-client

Related issues

closes #3807

Checklist

  • PR has a milestone or the no milestone label.

@@ -86,8 +86,8 @@
}

@SuppressWarnings({"rawtypes", "unchecked"})
public ZeebeFuture<ProcessInstanceEvent> send() {
ZeebeClientFutureImpl future = new ZeebeClientFutureImpl<>();
public CamundaFuture<ProcessInstanceEvent> send() {

Check notice

Code scanning / CodeQL

Missing Override annotation Note test

This method overrides
FinalCommandStep.send
; it is advisable to add an Override annotation.
@@ -140,7 +141,7 @@
new ProcessDefinitionVersion(
processDef.getProcessDefinitionKey(), processDef.getVersion()))));

var bpmnTest = ZeebeTest.with(zeebeClient).deploy(model).createInstance();
var bpmnTest = ZeebeTest.with(camundaClient).deploy(model).createInstance();

Check notice

Code scanning / CodeQL

Unread local variable Note test

Variable 'ZeebeTest bpmnTest' is never read.
@mathias-vandaele mathias-vandaele changed the title refactor(camunda-client): 1st commit, further investigation needed me… refactor(camunda-client): replace zeebe-client by camunda-client Jan 3, 2025
@mathias-vandaele mathias-vandaele marked this pull request as ready for review January 3, 2025 10:17
@mathias-vandaele mathias-vandaele requested a review from a team as a code owner January 3, 2025 10:17
@mathias-vandaele mathias-vandaele added this to the 8.7.0-alpha3 milestone Jan 3, 2025
@mathias-vandaele mathias-vandaele self-assigned this Jan 3, 2025
@github-actions github-actions bot temporarily deployed to connectors-3807-update-zee-c8sm January 6, 2025 10:26 Destroyed
@github-actions github-actions bot temporarily deployed to connectors-3807-update-zee-c8sm January 8, 2025 13:56 Destroyed
@mathias-vandaele mathias-vandaele force-pushed the 3807-update-zeebe-java-client-to-new-camunda-java-client-artifact-name-change branch from 19c270b to b494f5a Compare January 9, 2025 08:36
@github-actions github-actions bot temporarily deployed to connectors-3807-update-zee-c8sm January 9, 2025 08:37 Destroyed
@github-actions github-actions bot temporarily deployed to connectors-3807-update-zee-c8sm January 9, 2025 09:59 Destroyed
@sbuettner sbuettner modified the milestones: 8.7.0-alpha3, 8.7.0-alpha4 Jan 9, 2025
@github-actions github-actions bot temporarily deployed to connectors-3807-update-zee-c8sm January 9, 2025 16:02 Destroyed
@github-actions github-actions bot temporarily deployed to connectors-3807-update-zee-c8sm January 10, 2025 08:06 Destroyed
Map<String, Object> customProperties)
Map<String, Object> customProperties,
String processDefinitionId,
Long processInstanceKey)
Copy link
Collaborator

Choose a reason for hiding this comment

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

why is that not in main already?

connectors/http/rest/README.peb Outdated Show resolved Hide resolved
@github-actions github-actions bot temporarily deployed to connectors-3807-update-zee-c8sm January 10, 2025 09:07 Destroyed
@johnBgood johnBgood self-requested a review January 10, 2025 09:11
Copy link
Collaborator

@johnBgood johnBgood left a comment

Choose a reason for hiding this comment

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

Looks great thanks!

@mathias-vandaele mathias-vandaele added this pull request to the merge queue Jan 10, 2025
Merged via the queue into main with commit 611bcfe Jan 10, 2025
12 checks passed
@mathias-vandaele mathias-vandaele deleted the 3807-update-zeebe-java-client-to-new-camunda-java-client-artifact-name-change branch January 10, 2025 11:20
@johnBgood
Copy link
Collaborator

/backport

@sbuettner
Copy link
Contributor

Backport failed for release/8.7, because it was unable to cherry-pick the commit(s).

Please cherry-pick the changes locally and resolve any conflicts.

git fetch origin release/8.7
git worktree add -d .worktree/backport-3826-to-release/8.7 origin/release/8.7
cd .worktree/backport-3826-to-release/8.7
git switch --create backport-3826-to-release/8.7
git cherry-pick -x 611bcfe4df88e77e36a242b5e3f5dd12387ee270

johnBgood pushed a commit that referenced this pull request Jan 10, 2025
* refactor(camunda-client): 1st commit, further investigation needed merging

* refactor(camunda-client): fixing the issue, refactor is done

* refactor(camunda-client) bring a single camunda version

* refactor(camunda-client) bring a single camunda version 5

* refactor(camunda-client) bring a single camunda version 6

* Update connectors/http/rest/README.peb

Co-authored-by: Jonathan <[email protected]>

---------

Co-authored-by: Jonathan <[email protected]>
(cherry picked from commit 611bcfe)
johnBgood added a commit that referenced this pull request Jan 10, 2025
…) (#3842)

* refactor(camunda-client): 1st commit, further investigation needed merging

* refactor(camunda-client): fixing the issue, refactor is done

* refactor(camunda-client) bring a single camunda version

* refactor(camunda-client) bring a single camunda version 5

* refactor(camunda-client) bring a single camunda version 6

* Update connectors/http/rest/README.peb

Co-authored-by: Jonathan <[email protected]>

---------

Co-authored-by: Jonathan <[email protected]>
(cherry picked from commit 611bcfe)

Co-authored-by: Mathias Vandaele <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update Zeebe Java client to new Camunda Java client (artifact name change)
3 participants