Skip to content

Conversation

@ellipsis-dev
Copy link
Contributor

@ellipsis-dev ellipsis-dev bot commented Jun 21, 2025

This change addresses review comments left by @mrubens on PR #4983: Filter out additional bot and automated accounts from contributors list

⚠️ We couldn't build/test your project to verify our changes. Add a Dockerfile to significantly improve code quality.

Summary:

Update formatContributorsSection to exclude cursor and roomote logins in update-contributors.js.

Key points:

  • Behavior:
    • Update formatContributorsSection in update-contributors.js to exclude contributors with logins cursor and roomote.
    • Refactor filtering logic to use EXCLUDED_LOGIN_SUBSTRINGS and EXCLUDED_LOGIN_EXACTS arrays for better readability and maintainability.

You can configure Ellipsis to address comments with a direct commit or a side PR, see docs.


Something look wrong? If this Pull Request doesn't address the comments left on the above pull request, create a new PR review with more details. For more information, check the documentation.

Generated with ❤️ by ellipsis.dev

… and automated accounts from contributors list);
@ellipsis-dev ellipsis-dev bot requested a review from mrubens June 21, 2025 16:58
@ellipsis-dev ellipsis-dev bot requested review from cte and jr as code owners June 21, 2025 16:58
@dosubot dosubot bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Jun 21, 2025
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jun 21, 2025
@mrubens mrubens merged commit 4ed55d4 into cursor/exclude-cursor-and-roomote-from-contributors-76cc Jun 21, 2025
6 checks passed
@mrubens mrubens deleted the ellipsis/jrun_JtCqO5REFm8HDwZs_LcbA branch June 21, 2025 16:59
@github-project-automation github-project-automation bot moved this from Triage to Done in Roo Code Roadmap Jun 21, 2025
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap Jun 21, 2025
mrubens added a commit that referenced this pull request Jun 21, 2025
…st (#4983)

* Filter out additional bot and automated accounts from contributors list

* [Ellipsis] fix: exclude `cursor` and `roomote` logins in contributors section (#4984)

Co-authored-by: ellipsis-dev[bot] <65095814+ellipsis-dev[bot]@users.noreply.github.com>

---------

Co-authored-by: Cursor Agent <[email protected]>
Co-authored-by: ellipsis-dev[bot] <65095814+ellipsis-dev[bot]@users.noreply.github.com>
cte pushed a commit that referenced this pull request Jun 24, 2025
…st (#4983)

* Filter out additional bot and automated accounts from contributors list

* [Ellipsis] fix: exclude `cursor` and `roomote` logins in contributors section (#4984)

Co-authored-by: ellipsis-dev[bot] <65095814+ellipsis-dev[bot]@users.noreply.github.com>

---------

Co-authored-by: Cursor Agent <[email protected]>
Co-authored-by: ellipsis-dev[bot] <65095814+ellipsis-dev[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR has been approved by a maintainer size:S This PR changes 10-29 lines, ignoring generated files.

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants