Add community contributor credits across maester.dev#1999
Merged
Conversation
Celebrates the security experts behind every Maester test: - New /contributors page: profile cards with GitHub avatars, taglines, company/location (with country flags), social links, MVP badges linked to MVP profiles, and clickable stats. Ranked by a weighted score (authored x5 + sqrt-dampened improvements) with the newest and most recently active contributors highlighted in a side pane. - Static per-contributor pages at /contributors/<handle> listing the tests each person authored and improved. - Contributor byline on every generated test doc page crediting the original author and co-contributors with an avatar stack. - Attribution is derived automatically from git history at build time (single rename-tracking pass so file moves don't steal authorship), with suite-level overrides (EIDSCA credited to Thomas Naunheim) and per-test overrides available in attribution-overrides.yml. - Identity matching is fully automatic: GitHub noreply addresses, a GitHub API lookup for unknown commit emails (cached as SHA-256 hashes in email-aliases.json - no plaintext addresses stored), and git author names. Snapshot committed at src/data/contributors.json as a fallback for shallow clones. - contributors.yml registry for optional profile polish (name, tagline, company, socials, MVP profile URL, avatar override via static/img/contributors/). - Website build workflow: fetch full history and pass GH_TOKEN so attribution stays fresh on every deploy. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Contributor
|
Important Review skippedToo many files! This PR contains 420 files, which is 270 over the limit of 150. To get a review, narrow the scope: Upgrade to a paid plan to raise the limit. Usage-priced reviews support at most 300 files. ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (2)
📒 Files selected for processing (420)
You can disable this status message by setting the ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Not up to standards ⛔🔴 Issues
|
| Category | Results |
|---|---|
| ErrorProne | 1 medium |
| Security | 7 critical 11 high |
🟢 Metrics 179 complexity
Metric Results Complexity 179
NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.
Deploying maester with
|
| Latest commit: |
5da9961
|
| Status: | ⚡️ Build in progress... |
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
Summary
Maester's tests are written by community SMEs — this PR makes that a first-class part of the site.
New
/contributorspagepinLast: true.)/contributors/<handle>listing the tests they authored and improved.Test-page bylines
Every generated test doc now opens with a byline crediting the original author (ringed avatar) and co-contributors, linking to their contributor pages.
How attribution works
website/contributors/email-aliases.json(no plaintext addresses stored, machine-maintained); git author names cover the rest.website/contributors/contributors.ymlis optional polish: display name, tagline, company, socials, MVP profile URL, avatar override. Seeded from the MVP directory and GitHub profiles.attribution-overrides.yml.website/src/data/contributors.jsonas fallback for shallow clones; the build workflow now fetches full history and passesGH_TOKENso production stays fresh.Notes for reviewers
website/docs/tests/**is regenerated output (bylines + MT.1186); the interesting diff is inwebsite/scripts/,website/src/, andwebsite/contributors/.npm run buildon this branch fails on one pre-existing broken link (MT.1186→ its command doc, which only the CI command-reference job generates). CI builds regenerate the command reference first and pass.Test plan
npm run generate-test-docs— 398 pages, all attributed🤖 Generated with Claude Code