Skip to content
This repository has been archived by the owner on Oct 10, 2024. It is now read-only.

New annotation names and defaults #229

Closed
wants to merge 1 commit into from
Closed

New annotation names and defaults #229

wants to merge 1 commit into from

Conversation

falko
Copy link
Contributor

@falko falko commented Sep 27, 2022 •
edited
Loading

Based on experience of using Spring Zeebe, Bernd and I are proposing a refactoring of annotations names and their default values.

@CLAassistant
Copy link

CLAassistant commented Sep 27, 2022 •
edited
Loading

CLA assistant check
All committers have signed the CLA.

@falko falko marked this pull request as draft September 27, 2022 10:51
@falko
Copy link
Contributor Author

falko commented Sep 29, 2022

@berndruecker the solution architects are happy with the current state including keeping the slightly longer but more explicit @VariablesAsType

@falko
Copy link
Contributor Author

falko commented Sep 29, 2022

This example should change too: https:/camunda/camunda-platform-tutorials/blob/main/orchestrate-microservices/worker-java/src/main/java/io/camunda/getstarted/tutorial/Worker.java

@falko
Copy link
Contributor Author

falko commented Sep 29, 2022

@berndruecker stakeholders are fine, too.
@menski FYI

@berndruecker berndruecker mentioned this pull request Oct 12, 2022
Closed
@berndruecker
Copy link
Contributor

Moved to #239

@berndruecker berndruecker deleted the annotation-renaming branch November 22, 2022 14:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@berndruecker berndruecker berndruecker left review comments

@chDame chDame chDame left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@falko @CLAassistant @berndruecker @chDame