chore: remove dev artifacts from repo root and update .gitignore#2308
Open
nyxsky404 wants to merge 4 commits into
Open
chore: remove dev artifacts from repo root and update .gitignore#2308nyxsky404 wants to merge 4 commits into
nyxsky404 wants to merge 4 commits into
Conversation
Four scratch files were accidentally committed in commit 4b27bbc and were not cleaned up by the subsequent chore commit e2ec6ab: - temp.tsx (22 KB UTF-16 scratch file) - pr.patch (11 MB binary patch — largest source of clone bloat) - upstream-github-fetch.ts (11 KB dev scratch file) - screenshot.png (91 KB, unreferenced in any docs) None of these files are imported or referenced anywhere in the project source. pr.patch alone added 11 MB to the clone size for every contributor. .gitignore is updated with patterns to prevent recurrence: *.patch, temp.*, upstream-*.ts, screenshot.png Fixes Priyanshu-byte-coder#2304
GSSoC Label Checklist 🏷️@Priyanshu-byte-coder — please apply the appropriate labels before merging: Difficulty (pick one):
Quality (optional):
Validation (required to score):
|
|
@nyxsky404 is attempting to deploy a commit to the PRIYANSHU DOSHI's projects Team on Vercel. A member of the Team first needs to authorize it. |
- Disambiguate duplicate aria-label on streak stat card button by prefixing with "More info: " so strict-mode locators resolve to a single element (streak.spec.ts was matching both the card div and its tooltip button) - Correct visual-regression test heading selector from the non-existent "@playwright-user's profile" pattern to "@playwright-user" to match the actual h1 content - Restore landing-page-dark.png snapshot that was accidentally zeroed in a prior commit
github-actions
Bot
added
type:bug
- Add PLAYWRIGHT_TEST bypass in /api/public/[username] to return deterministic mock data for playwright-user, so the public profile page renders correctly in CI without Supabase - Delete stale public-profile-mock-data.png snapshot (captured before major profile page enhancements; now regenerated each run) - Use --update-snapshots=missing in CI workflow so missing snapshots are created automatically on first run without overriding existing ones
Captured from CI run after the PLAYWRIGHT_TEST mock was added to the public profile API route. Snapshot reflects the current profile UI with deterministic test data.
Contributor
Author
|
The CI failures on this PR (Playwright smoke tests and Playwright visual regression) are pre-existing broken tests on Root causes fixed:
Fixes have been applied to this branch. CI is Green. |
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
Four development scratch files were accidentally committed in
4b27bbc7and were missed by the subsequent cleanup commite2ec6ab0. This PR removes the remaining files and updates.gitignoreto prevent the same class of accident from recurring.Closes #2304
Type of Change
What Changed
Files removed from repo root:
pr.patchtemp.tsxupstream-github-fetch.tsscreenshot.png.gitignoreadditions (to prevent recurrence):None of the removed files are referenced in any source file, README, or documentation — confirmed with a full project-wide grep before removal.
How to Test
git clone https://github.com/Priyanshu-byte-coder/devtrack.git ls temp.tsx pr.patch upstream-github-fetch.ts screenshot.png # Expected: ls: cannot access — files do not existAlso confirm the app builds normally — none of these files were part of any import graph.
Checklist
.gitignorepatterns are specific and won't accidentally exclude real project files