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

[#13164] Fix typo #13165

Merged
merged 3 commits into from
Feb 16, 2025
Merged

[#13164] Fix typo #13165

merged 3 commits into from
Feb 16, 2025

Conversation

itstrueitstrueitsrealitsreal
Copy link
Contributor

Fixes #13164

@nusoss-bot
Copy link

Folks, This PR seems to be stalling (no activities for the past 7 days). 🐌 😢
Hope someone can get it to move forward again soon...

1 similar comment
@nusoss-bot
Copy link

Folks, This PR seems to be stalling (no activities for the past 7 days). 🐌 😢
Hope someone can get it to move forward again soon...

@nusoss-bot
Copy link

Folks, This PR seems to be stalling (no activities for the past 9 days). 🐌 😢
Hope someone can get it to move forward again soon...

weiquu
weiquu approved these changes Sep 2, 2024
View reviewed changes
Copy link
Contributor

@weiquu weiquu left a comment •
edited
Loading

Choose a reason for hiding this comment

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

LGTM, thanks for the catch (edit: will merge after #13173 is fixed)

@weiquu weiquu added the s.ToMerge The PR is approved by all reviewers including final reviewer; ready for merging label Sep 2, 2024
@itstrueitstrueitsrealitsreal itstrueitstrueitsrealitsreal mentioned this pull request Feb 11, 2025
Merged
jasonqiu212
jasonqiu212 approved these changes Feb 15, 2025
View reviewed changes
Copy link
Contributor

@jasonqiu212 jasonqiu212 left a comment

Choose a reason for hiding this comment

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

LGTM!

@jasonqiu212 jasonqiu212 merged commit d7e82bd into TEAMMATES:master Feb 16, 2025
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jasonqiu212 jasonqiu212 jasonqiu212 approved these changes

@weiquu weiquu weiquu approved these changes

Assignees
No one assigned
Labels
s.ToMerge The PR is approved by all reviewers including final reviewer; ready for merging
Projects
None yet
Milestone
No milestone
4 participants
@itstrueitstrueitsrealitsreal @nusoss-bot @jasonqiu212 @weiquu