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

shorten the text in the comment, to make less annoying #137

Open
aladdin-add opened this issue Jan 22, 2021 · 6 comments
Open

shorten the text in the comment, to make less annoying #137

aladdin-add opened this issue Jan 22, 2021 · 6 comments

Comments

@aladdin-add
Copy link
Member

aladdin-add commented Jan 22, 2021

e.g. eslint/eslint#14010 (comment)

@nzakas
Copy link
Member

nzakas commented Jan 23, 2021

Sorry, I’m not sure what it is you’re asking. Can you explain?

@aladdin-add
Copy link
Member Author

@anikethsaha
Copy link
Member

agreed that these comments are making a bit of noise.

But if we are editing the existing comment, then the author won't get notified with the latest status of its commit right !

@nzakas
Copy link
Member

nzakas commented Jan 26, 2021

@aladdin-add thanks. In the future, please be sure to include a full description when opening an issue.

I also don’t think editing the comment makes much sense. I would like to shorten the text in the comment, though, as I think that would make it less annoying.

@aladdin-add aladdin-add changed the title editing the existing comment, than creating a new one. shorten the text in the comment, to make less annoying Jan 27, 2021
@mdjermanovic
Copy link
Member

just like in the PR eslint/eslint#14010, the bot commented 3 times:

it makes a little harder to read, I think it's better not to create multiple comments.

Thoughts about deleting the previous comment when the new one should be added?

@nzakas
Copy link
Member

nzakas commented Feb 9, 2021

I think it’s better to just not comment more than once.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants