Skip to content

improve: workflow builder naming #2559

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

Merged
merged 3 commits into from
Oct 26, 2024
Merged

improve: workflow builder naming #2559

merged 3 commits into from
Oct 26, 2024

Conversation

csviri
Copy link
Collaborator

@csviri csviri commented Oct 25, 2024

No description provided.

@csviri csviri requested a review from metacosm October 25, 2024 11:03
@openshift-ci openshift-ci bot requested a review from adam-sandor October 25, 2024 11:03
Signed-off-by: Attila Mészáros <[email protected]>
@csviri csviri changed the title improve: use the same naming as in annotation for in workflow builder improve: workflow builder naming Oct 25, 2024
@csviri
Copy link
Collaborator Author

csviri commented Oct 25, 2024

@metacosm a bit nit on naming, let me know if sounds better also to you ears :)

Signed-off-by: Attila Mészáros <[email protected]>
@csviri
Copy link
Collaborator Author

csviri commented Oct 25, 2024

Was thining about more variation, I know how did you mean it but, just having the same wording as in annotations (where it makes sense) is a bit better IMO.

(

)

@metacosm
Copy link
Collaborator

I prefer the fluent version but using the same wording as the annotation attribute makes sense.

@csviri csviri merged commit 81d0773 into next Oct 26, 2024
57 checks passed
@csviri csviri deleted the naming-dr-build branch October 26, 2024 11:21
metacosm pushed a commit that referenced this pull request Nov 5, 2024
metacosm pushed a commit that referenced this pull request Nov 6, 2024
csviri added a commit that referenced this pull request Nov 13, 2024
metacosm pushed a commit that referenced this pull request Nov 19, 2024
metacosm pushed a commit that referenced this pull request Nov 20, 2024
metacosm pushed a commit that referenced this pull request Nov 27, 2024
csviri added a commit that referenced this pull request Dec 6, 2024
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