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.

Sign up for GitHub

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

Jump to bottom

Updated the GUI pin for 2.5.1 #19158

Open
wants to merge 1 commit into
base: release/2.5.1
Choose a base branch
from
Open

Updated the GUI pin for 2.5.1 #19158

wants to merge 1 commit into from

Conversation

ChiaMineJP
Copy link
Contributor

@ChiaMineJP ChiaMineJP commented Jan 17, 2025 •
edited
Loading

@ChiaMineJP ChiaMineJP added the Changed Required label for PR that categorizes merge commit message as "Changed" for changelog label Jan 17, 2025
@ChiaMineJP ChiaMineJP requested a review from emlowe January 17, 2025 16:21
@ChiaMineJP ChiaMineJP self-assigned this Jan 17, 2025
@ChiaMineJP ChiaMineJP requested a review from a team as a code owner January 17, 2025 16:21
@ChiaMineJP ChiaMineJP marked this pull request as draft January 18, 2025 05:39
@ChiaMineJP ChiaMineJP force-pushed the cmj/update-gui-pin-20250118 branch from 570a13e to 331268b Compare January 22, 2025 14:57
@ChiaMineJP ChiaMineJP marked this pull request as ready for review January 22, 2025 14:58
wjblanke
wjblanke approved these changes Jan 22, 2025
View reviewed changes
Copy link
Contributor

@wjblanke wjblanke left a comment

Choose a reason for hiding this comment

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

aok

@AmineKhaldi
Copy link
Contributor

@ChiaMineJP it would help if you could kindly mention the pinned commit in the description, for checkpointing purposes. Thanks!

@ChiaMineJP
Copy link
Contributor Author

ChiaMineJP commented Jan 22, 2025 •
edited
Loading

@ChiaMineJP it would help if you could kindly mention the pinned commit in the description, for checkpointing purposes. Thanks!

@AmineKhaldi
Sorry, the pin is set to this one: a3b94b23a6963238bffe8fce251f4dd4cc1819cc

@AmineKhaldi AmineKhaldi mentioned this pull request Jan 23, 2025
Draft
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@markelrod markelrod markelrod approved these changes

@wjblanke wjblanke wjblanke approved these changes

@emlowe emlowe Awaiting requested review from emlowe

Assignees

@ChiaMineJP ChiaMineJP

Labels
Changed Required label for PR that categorizes merge commit message as "Changed" for changelog
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@ChiaMineJP @AmineKhaldi @wjblanke @markelrod