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

renable contrib badge bot, with optional commenting #656

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

nathanwoulfe
Copy link

As part of the community teams catchup last week, we identified a few areas for improving the core contributions experience - one of which being automatically adding the contrib badge to Our profiles. Despite several in attendance having also been at the 2019 retreat, we forgot that this had been tackled already.

The automation code was disabled by a feature flag, most likely because at the time it was added, few Our profiles would have had the GitHub account linked. That's no longer the case, so we figured it's safe to turn it back on. To make it a bit safer, I've added an additional app setting to disable comments - this means the bot can be enabled silently, if we want to turn it on and monitor it for a while, without firing off comments to PRs.

For that to work, the web config needs two new app settings added (file is excluded from source control). Keys are:

<add key="EnableContribBadgeBot" value="#{EnableContribBadgeBot}#" />
<add key="EnableContribBadgeBotComments" value="#{EnableContribBadgeBotComments}#" />

Everything else is largely unchanged - I tweaked the comment text a little and updated a couple of variable names, nothing major. Added a null-check in MarkAsSolutionReminder because it kept throwing errors on local.

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.

1 participant