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

fix: Translate job-group properly #303

Merged
merged 1 commit into from
Dec 10, 2024
Merged

Conversation

skycastlelily
Copy link
Contributor

@skycastlelily skycastlelily commented Dec 4, 2024

No description provided.

@skycastlelily
Copy link
Contributor Author

Block tmt mr: teemtee/tmt#3397 :)

@dav-pascual dav-pascual self-assigned this Dec 5, 2024
@dav-pascual dav-pascual self-requested a review December 5, 2024 12:13
@dav-pascual dav-pascual removed their assignment Dec 5, 2024
@skycastlelily
Copy link
Contributor Author

Hi Dav, thanks for your review, I have updated the commit message, I guess we are good now, and sorry for failing to note that before I submit this mr 😅

@dav-pascual
Copy link
Member

Hi @skycastlelily
Thank you for your changes. In principle LGTM.
The reason behind this change is that previous key was wrong, right?

@dav-pascual dav-pascual changed the title Translate job-group properly fix: Translate job-group properly Dec 5, 2024
@skycastlelily
Copy link
Contributor Author

The reason behind this change is that previous key was wrong, right?

Yes ,
BeakerJob use job-group key instead of group.
I have moved the above line here,hope we can get rid of the PR header format error:)

@dav-pascual
Copy link
Member

LGTM. Nevermind about the failing check, idk why it's still failing

@dav-pascual dav-pascual merged commit 0cabc75 into neoave:main Dec 10, 2024
14 of 15 checks passed
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