Clarify how to be added to allow list #16
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
On first reading, I found very confusing to understand how I could be added to allow list. I just tried creating the PR anyway and discovered that the first message by your bot explains how to add someone. I believe that it would improve first user interaction if they already knew how to find this user that may allow them. (Even better would be to have it automated, but that's a more complex task I don't dare to try to work on)
The reason it is not automated is that it is a crude (but so far effective) anti-spam measure. GitGitGadget could be modified to allow everybody to |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
On first reading, I found very confusing to understand how I could be added to allow list. I just tried creating the PR anyway and discovered that the first message by your bot explains how to add someone. I believe that it would improve first user interaction if they already knew how to find this user that may allow them. (Even better would be to have it automated, but that's a more complex task I don't dare to try to work on)
Clone of #14, applying required correction. And this time I figured out what I got wrong which caused it to be closed. Very sorry for the noise