Skip to content

expand on edstem#23

Merged
phrdang merged 3 commits into
mainfrom
ed
Jun 29, 2025
Merged

expand on edstem#23
phrdang merged 3 commits into
mainfrom
ed

Conversation

@pancakereport
Copy link
Copy Markdown
Collaborator

@pancakereport pancakereport commented Jun 4, 2025 •
edited by phrdang
Loading

  • lock threads during exams
  • private posts @seany4ng you mentioned wanting this and i haven't been a TA recently enough to provide my own opinionated guidance

@pancakereport pancakereport marked this pull request as draft June 4, 2025 02:16
@phrdang
Copy link
Copy Markdown
Member

phrdang commented Jun 6, 2025

Not Sean, but what you wrote is basically all a TA would need to know in my opinion

@seany4ng
Copy link
Copy Markdown
Contributor

seany4ng commented Jun 6, 2025

Yep +1 on what Rebecca mentioned above!

@phrdang phrdang marked this pull request as ready for review June 29, 2025 22:37
phrdang
phrdang approved these changes Jun 29, 2025
View reviewed changes
Copy link
Copy Markdown
Member

@phrdang phrdang left a comment

Choose a reason for hiding this comment

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

Thanks LGTM!

@phrdang phrdang merged commit 0681912 into main Jun 29, 2025
5 checks passed
@phrdang phrdang deleted the ed branch June 29, 2025 22:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@phrdang phrdang phrdang approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@pancakereport @phrdang @seany4ng