Skip to content

feat(Seer): Updated Seer active contributor notes to include Gitlab#18669

Open
sfanahata wants to merge 1 commit into
masterfrom
feat/sanahata/gitlab-account-charges
Open

feat(Seer): Updated Seer active contributor notes to include Gitlab#18669
sfanahata wants to merge 1 commit into
masterfrom
feat/sanahata/gitlab-account-charges

Conversation

@sfanahata

Copy link
Copy Markdown
Contributor

DESCRIBE YOUR PR

Includes details about how active contributors work for Gitlab.

IS YOUR CHANGE URGENT?

Help us prioritize incoming PRs by letting us know when the change needs to go live.

  • Urgent deadline (GA date, etc.): ASAP
  • Other deadline:
  • None: Not urgent, can wait up to 1 week+

SLA

  • Teamwork makes the dream work, so please add a reviewer to your PRs.
  • Please give the docs team up to 1 week to review your PR unless you've added an urgent due date to it.
    Thanks in advance for your help!

PRE-MERGE CHECKLIST

Make sure you've checked the following before merging your changes:

  • Checked Vercel preview for correctness, including links
  • PR was reviewed and approved by any necessary SMEs (subject matter experts)
  • PR was reviewed and approved by a member of the Sentry docs team

@vercel

vercel Bot commented Jul 7, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
sentry-docs Ready Ready Preview, Comment Jul 7, 2026 11:40pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
develop-docs Ignored Ignored Jul 7, 2026 11:40pm

Request Review

@sfanahata sfanahata requested a review from billyvg July 7, 2026 23:30
Comment thread docs/pricing/quotas/manage-seer-budget.mdx

<Alert>
GitLab does not support the GitHub Checks API. Code Review results appear as MR comments, not as a pipeline status check. You cannot use Seer as a required GitLab CI/CD check.
GitLab does not support external status checks. Code Review results appear as MR comments, not as a pipeline status check. You cannot use Seer as a required GitLab CI/CD check.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

actually i think we can get rid of this noptice completely as we do support pipelines.

<Alert level="warning">
All members of your repositories, *except* GitHub bots marked as `[bot]`, are counted as active contributors.

For Gitlab, all accounts (group members) are counted as active contributors, even when using Autofix or bots for fixes.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

so actually we ignore the following gitlab username patterns:

project access token bots — project_{id}_bot_…
group access token bots — group_{id}_bot_…
service accounts — service_account_… / service_account_group_{id}_…

these aren't counted as contributors.

what is counted as a contribution are PRs that are created by Seer. for github, these prs are created by the Sentry GitHub app, in gitlab it is created by the user account that was used when integrating GitLab in Sentry.

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