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 a comment asking for feedback #40

Merged
merged 4 commits into from
Oct 5, 2022
Merged

Add a comment asking for feedback #40

merged 4 commits into from
Oct 5, 2022

Conversation

jiegillet
Copy link
Contributor

@jiegillet jiegillet commented Oct 3, 2022

Closes #37.

Sister PR here.

If there are any essential or actionable comments, we automatically add an extra comment to ask for feedback.
If there is no comment, or if there are only informative or celebratory comments, we don't add anything.

This is still in draft because I need to fix the smoke tests, but they were changed in #39 so I would need to redo it anyway.

@jiegillet jiegillet force-pushed the jie-feedback-comment branch from 996c65a to 8fcb75e Compare October 4, 2022 12:58
@jiegillet jiegillet marked this pull request as ready for review October 4, 2022 12:59
@jiegillet jiegillet requested a review from a team as a code owner October 4, 2022 12:59
@jiegillet
Copy link
Contributor Author

@exercism/maintainers-admin would you approve this PR please?

@jiegillet jiegillet merged commit 8ecc3a6 into main Oct 5, 2022
@jiegillet jiegillet deleted the jie-feedback-comment branch November 9, 2023 00:35
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.

Add a comment asking for feedback
3 participants