-
Notifications
You must be signed in to change notification settings - Fork 31
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
[FEATURE] Create a new Util method for a common pattern #720
Labels
Comments
owaiskazi19
added
enhancement
New feature or request
good first issue
Good for newcomers
untriaged
labels
May 30, 2024
@martinpkr can you comment on this issue so we can assign it to you? |
Will do in 20-25 minutes!
…On Thu, 30 May 2024 at 18:51, Daniel Widdis ***@***.***> wrote:
@martinpkr <https://github.com/martinpkr> can you comment on this issue
so we can assign it to you?
—
Reply to this email directly, view it on GitHub
<#720 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AU52WT6PESGW5MFEPS7BICDZE5DI7AVCNFSM6AAAAABIP7IAPCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBQGAZDQMJRGM>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
You just did, haha! |
Didn’t know it worked like that, awesome haha
…On Thu, 30 May 2024 at 18:56, Daniel Widdis ***@***.***> wrote:
Will do in 20-25 minutes!
You just did, haha!
—
Reply to this email directly, view it on GitHub
<#720 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AU52WTYILORUH44HS65CEO3ZE5D33AVCNFSM6AAAAABIP7IAPCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBQGA2DKNRWGY>
.
You are receiving this because you were assigned.Message ID:
***@***.***>
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Is your feature request related to a problem?
We have the below pattern in couple of WorkflowSteps in the codebase
This can be refactored into a util method to avoid duplication.
What solution would you like?
Create a Util method in ParseUtils.java file, have the above code in it. Invoke the util method everywhere where the above pattern is found.
What alternatives have you considered?
Keep it as it is
Do you have any additional context?
Add any other context or screenshots about the feature request here.
The text was updated successfully, but these errors were encountered: