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

Auto-comment on submitted PRs #7248

Merged
merged 5 commits into from
Apr 29, 2020
Merged

Auto-comment on submitted PRs #7248

merged 5 commits into from
Apr 29, 2020

Conversation

hsanjuan
Copy link
Contributor

  • Give a few tips on linking relevant discussions, self reviews, additional
    reviews and code contribution guidelines
  • Be explicit about the next steps and what to expect.

I would like to trial this. A potential problem I see is that this will comment on every PR (including ours) and might be annoying for the people that sends most PRs. We can switch to commenting on only first-time PRs but then we miss the chance to remind people that only occasionally contribute (and ourselves about the process to follow).

* Give a few tips on linking relevant discussions, self reviews, additional
  reviews and code contribution guidelines
* Be explicit about the next steps and what to expect.
@hsanjuan hsanjuan self-assigned this Apr 29, 2020
@ribasushi
Copy link
Contributor

@hsanjuan is it possible to give this some sort of predefined Subject or somesuch, so that one can auto-filter these out? Our mailboxes are overloaded as it is...

@hsanjuan
Copy link
Contributor Author

@hsanjuan is it possible to give this some sort of predefined Subject or somesuch, so that one can auto-filter these out? Our mailboxes are overloaded as it is...

You would be able to filter out the botname (even block it on github so you don't even see these).

Copy link
Member

@Stebalien Stebalien left a comment

Choose a reason for hiding this comment

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

This is a lot of text for an automatic comment.

.github/auto-comment.yml Outdated Show resolved Hide resolved
.github/auto-comment.yml Outdated Show resolved Hide resolved
.github/auto-comment.yml Outdated Show resolved Hide resolved
@hsanjuan
Copy link
Contributor Author

This is a lot of text for an automatic comment.

I know. I've done another pass trying to be more concise. Can you check?

@Stebalien Stebalien merged commit 54a0da5 into master Apr 29, 2020
@Stebalien Stebalien mentioned this pull request May 26, 2020
77 tasks
@hacdias hacdias deleted the feat/pr-autocomment branch May 9, 2023 10:58
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.

3 participants