Skip to content

Add PR commenter #1445

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

Merged
merged 17 commits into from
Oct 17, 2021
Merged

Add PR commenter #1445

merged 17 commits into from
Oct 17, 2021

Conversation

junedev
Copy link
Member

@junedev junedev commented Oct 15, 2021

Resolves #1436

The final result in the case that all possible comments are present can be seen here: junedev#5 (comment)

This should be seen as a first iteration. Ideas for more items in the list should be discussed in a new issue, not as part of the PR.

Putting each sentence on a separate line did not work here, as it rendered as such and looked weird.

@junedev junedev marked this pull request as ready for review October 17, 2021 11:15
@junedev junedev requested a review from a team October 17, 2021 11:15
@junedev junedev added the x:size/medium Medium amount of work label Oct 17, 2021
Copy link
Member

@SleeplessByte SleeplessByte left a comment

Choose a reason for hiding this comment

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

Great!

@SleeplessByte SleeplessByte added the hacktoberfest-accepted Opt-in to hacktoberfest label Oct 17, 2021
junedev and others added 2 commits October 17, 2021 16:49
Co-authored-by: Derk-Jan Karrenbeld <derk-jan+github@karrenbeld.info>
@junedev junedev merged commit 78cbe38 into exercism:main Oct 17, 2021
@junedev junedev deleted the add-pr-commenter branch October 17, 2021 14:56
pertrai1 pushed a commit to pertrai1/javascript that referenced this pull request Oct 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hacktoberfest-accepted Opt-in to hacktoberfest x:size/medium Medium amount of work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add an automated PR comment
2 participants