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

Move namespaced to the internal structure #747

Merged
merged 1 commit into from
Dec 10, 2024

Conversation

ygalblum
Copy link
Contributor

Type of change

  • Refactor

Description

The field Namespaced of the external Object is never passed by the user and always gets set internally. Therefore, the field belongs to the internal object structure and can be renamed to namespaced

@ygalblum ygalblum force-pushed the namespaced-internal branch 2 times, most recently from 9af9791 to 664f132 Compare November 26, 2024 19:24
@rsevilla87 rsevilla87 self-requested a review December 2, 2024 16:30
@rsevilla87
Copy link
Member

Can you rebase please?

Signed-off-by: Ygal Blum <ygal.blum@gmail.com>
@ygalblum ygalblum force-pushed the namespaced-internal branch from 664f132 to e767676 Compare December 2, 2024 20:48
@ygalblum
Copy link
Contributor Author

ygalblum commented Dec 2, 2024

Can you rebase please?

Done

Copy link
Member

@rsevilla87 rsevilla87 left a comment

Choose a reason for hiding this comment

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

lgtm

@rsevilla87 rsevilla87 merged commit 42c9cb1 into kube-burner:main Dec 10, 2024
7 checks passed
@ygalblum ygalblum deleted the namespaced-internal branch December 10, 2024 17:10
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