eslint-github-bot icon indicating copy to clipboard operation
eslint-github-bot copied to clipboard

shorten the text in the comment, to make less annoying

Open aladdin-add opened this issue 4 years ago • 6 comments

e.g. https://github.com/eslint/eslint/pull/14010#issuecomment-761738132

aladdin-add avatar Jan 22 '21 03:01 aladdin-add

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

nzakas avatar Jan 23 '21 01:01 nzakas

just like in the PR https://github.com/eslint/eslint/pull/14010, the bot commented 3 times:

  • https://github.com/eslint/eslint/pull/14010#issuecomment-761737961
  • https://github.com/eslint/eslint/pull/14010#issuecomment-761738076
  • https://github.com/eslint/eslint/pull/14010#issuecomment-761738132

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

aladdin-add avatar Jan 23 '21 01:01 aladdin-add

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 !

anikethsaha avatar Jan 23 '21 07:01 anikethsaha

@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.

nzakas avatar Jan 26 '21 01:01 nzakas

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?

mdjermanovic avatar Jan 29 '21 20:01 mdjermanovic

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

nzakas avatar Feb 09 '21 01:02 nzakas