Repository navigation
chore: keep only the contributor collector - #461
Merged
Merged
Conversation
The leaderboard and the profiles moved to nuxt.com/nuxters, which imports the files in public/. Remove the Nuxt app and serve nuxters.nuxt.com as a static deployment of public/: the redirects and the JSON files keep working.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Deploying nuxters with
|
| Latest commit: |
c13fb8b
|
| Status: | 🚫 Build failed. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info
📝 Walkthrough
Merge Risk: ⚪ Minimal · up to
|
| Check name | Status | Explanation |
|---|---|---|
| Title check |  |
The title clearly summarizes the primary change: removing the Nuxt app and retaining the contributor collector. |
| Description check | |
The description directly explains the removal of the Nuxt app, the retained collector and public files, static deployment changes, tooling updates, and validation steps. |
| Docstring Coverage | |
No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 1… |
| Linked Issues check | |
Check skipped because no linked issues were found for this pull request. |
| Out of Scope Changes check | |
Check skipped because no linked issues were found for this pull request. |
Full details: Docstring Coverage
-
Explanation
No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 1 files. (5 skipped: 5 unsupported.)
✨ Finishing Touches
-
📝 Generate docstrings
-
- Commit to this branch
- Create a new PR
🧪 Generate unit tests (beta)
-
- Commit to this branch
- Create a new PR
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
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 @coderabbitai help to get the list of available commands.
This branch was successfully deployed
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
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.
The leaderboard and the profiles moved to nuxt.com/nuxters, which imports the files in
public/on every push (nuxt/nuxt.com#2458). The Nuxt app here no longer serves any page since #455, so this PR keeps only what the collector and nuxters.nuxt.com still need.Removed:
app/,server/,shared/,nuxt.config.ts, the Nuxt component and browser tests, the placeholder unit test,.env.example(OAuth and Discord secrets of the old app), the card and background images, and about 30 dependencies.Kept: the collector (
scripts/), its country tests (test/unit/countries.spec.ts, from #460),public/contributors*.json,public/profiles.json, the workflows and the Renovate config.nuxters.nuxt.com becomes a static deployment of
public/:vercel.jsonsets no framework, no install, no build andoutputDirectory: "public". The redirects are unchanged./contributors.jsonis still served, since at leastzhcndoc/nuxtandniceplugin/nuxt-koreanfetch it./api/contributorsis rewritten to/contributors.json;/api/contributors/<username>now returns 404 (no callers found in GitHub code search).Tooling: standalone
tsconfig.jsonandeslint.config.mjs(@nuxt/eslint-config), scripts arelint,typecheck,testandcollect:contributors, and CI runs all three checks.pnpm lint,pnpm typecheckandpnpm test(20 tests) pass locally, andCOLLECT_REPOS=nuxt/nuxters pnpm collect:contributorswrote every period file andprofiles.json. To check on the Vercel preview before merging: the redirects answer 301 and/contributors.jsonand/api/contributorsanswer 200.Written by an AI agent (Claude Code) on behalf of @atinux, who asked for this change.