Skip to content

feat: add bots configuration to display user activity#49

Merged
bjohansebas merged 2 commits intomainfrom
bots-elimite
Jan 7, 2026
Merged

feat: add bots configuration to display user activity#49
bjohansebas merged 2 commits intomainfrom
bots-elimite

Conversation

@bjohansebas
Copy link
Member

I’m adding a configuration option to hide bot accounts. I’m making it optional because removing all those accounts outright would be a breaking change. In the future, when we release a major version, we can make excluding bot-type accounts from the count the default behavior.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds a configuration option to control whether bot accounts are included in the user activity statistics. The feature is opt-in to maintain backward compatibility, with bots included by default.

  • Added bots configuration parameter with default value of true
  • Implemented filtering logic to exclude bot accounts based on GitHub API type field and login pattern
  • Exposed bots configuration to client-side templates

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

File Description
lib/config.js Adds the bots configuration option with a default value of true to include bots in statistics
template/index.html Passes the bots configuration value to the client-side JavaScript
template/js/index.js Implements filtering logic to exclude bot accounts from user activity list based on type field and login pattern

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@bjohansebas bjohansebas merged commit 48c0cd0 into main Jan 7, 2026
10 checks passed
@bjohansebas bjohansebas deleted the bots-elimite branch January 7, 2026 17:55
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.

2 participants