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

Add homersimpsons (myself) to PHP automator #2270

Merged
merged 1 commit into from
Sep 28, 2023
Merged

Add homersimpsons (myself) to PHP automator #2270

merged 1 commit into from
Sep 28, 2023

Conversation

homersimpsons
Copy link
Contributor

I did build https://github.com/exercism/php-representer

I also do mentoring on the JS track, but I guess it has more attention.

Copy link
Member

@IsaacG IsaacG left a comment

Choose a reason for hiding this comment

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

On the one hand, you did build it. On the other, you've only mentored 25 students. I would highly recommend going through a few more mentoring sessions prior to adding any automated feedback. Having experience with different students and approaches can be helpful when it comes to writing one-off feedback.

@homersimpsons
Copy link
Contributor Author

homersimpsons commented Sep 28, 2023

Thank you for the approval @IsaacG

On the other, you've only mentored 25 students

I did mentor 39 solutions but at first I was closing the mentoring sessions when the student send me a "thank you message" and these didn't count. I also closed some without answers after months to cleanup my workspave (it looks like since yesterday those get now closed with timeout).

I would highly recommend going through a few more mentoring sessions prior to adding any automated feedback

I will be careful with the tool. At first the idea is more to check how the representer behaves, in fact I have a local setup to download the first 500 submissions then run them through the representer and open only the different ones to see if there is any change I can do to the representer, but having this directly integrated in Exercism feel way better.

Having experience with different students and approaches can be helpful when it comes to writing one-off feedback.

I agree

@IsaacG IsaacG merged commit 7c0ea27 into exercism:main Sep 28, 2023
1 check 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