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

Allow commenting to be turned off #253

Open
MorrisonCole opened this issue Feb 5, 2022 · 7 comments
Open

Allow commenting to be turned off #253

MorrisonCole opened this issue Feb 5, 2022 · 7 comments
Assignees
Labels
feature New feature

Comments

@MorrisonCole
Copy link
Owner

I guess some users will just want the status check and nothing else - it can be annoying to get a tonne of comments!

@MorrisonCole MorrisonCole self-assigned this Feb 5, 2022
@MorrisonCole MorrisonCole added the feature New feature label Feb 5, 2022
@Couto
Copy link

Couto commented Feb 24, 2022

I would add that a better solution was to, either:

  • Dont add comments again (it is annoying to have 20 comments stating the same thing)
  • Update the same comment, I prefer this solution honestly, as the same comment can be edited after the action is successful.

@MorrisonCole
Copy link
Owner Author

@Couto yep to be honest didn't give it much thought. I'll have a look at updating the comment / preventing multiple comments when I get around to this!

@gregorybolkenstijn
Copy link

I would love any of the above mentioned options! 🔥

@thedmeyer
Copy link

@MorrisonCole Would love to see this! Any updates?

@gregorybolkenstijn
Copy link

@MorrisonCole Any chance this PR could be merged? #286

@MorrisonCole
Copy link
Owner Author

Hey folks - I'm really sorry for having this repo on the backburner for such a long time. I have of course seen all your comments and have set some time aside to address all the open issues this weekend. I'll ping all the threads once the released versions are out!

@MorrisonCole
Copy link
Owner Author

v1.7.0 is now released and adds the approach suggested by @Couto of updating the existing comment. Since I'd still like to add the option to turn off comments entirely, I'm leaving this issue open for now, but the annoying multiple-comments problem is at least fixed for now :)

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

No branches or pull requests

4 participants