fix: remove leftover debug console.log from useVideoEditor#703
fix: remove leftover debug console.log from useVideoEditor#703vivekyadav-3 wants to merge 1 commit into
Conversation
|
@vivekyadav1207vy-sudo is attempting to deploy a commit to the magic-peach1's projects Team on Vercel. A member of the Team first needs to authorize it. |
✅ PR Format Check Passed — @vivekyadav-3Basic format checks passed. A maintainer will review your code changes. This does not mean the PR is approved — it just means the format is correct. |
👋 Thanks for your PR, @vivekyadav-3!Welcome to Reframe — a browser-based video editor built for everyone 🎬
What happens next
Quick checklist
Useful links
Happy coding! 🎉 |
There was a problem hiding this comment.
Pull request overview
Removes a leftover debug console.log inside the development-only memory monitor effect in useVideoEditor.ts, replacing it with a placeholder comment to satisfy the CONTRIBUTING.md "no console.log" rule.
Changes:
- Replaces the
console.log("[Reframe Memory]", ...)statement with a// Track memory usage if neededcomment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| const mem = (performance as Performance & { memory?: { usedJSHeapSize: number } }).memory; | ||
| if (mem) { | ||
| console.log("[Reframe Memory]", Math.round(mem.usedJSHeapSize / 1e6), "MB used"); | ||
| // Track memory usage if needed |
a7a1234 to
24071c1
Compare
Description
Removed a leftover debug
console.logfromuseVideoEditor.ts(line 258) that was tracking memory usage. This ensures adherence to the rule defined inCONTRIBUTING.md("No console.log: Remove debug logs before submitting").Related Issue
None (Minor Code Cleanup)
Type of Contribution
Participant Info
Screen Recording
Recording / Loom link: N/A - This is a pure code-cleanup PR with no UI/design changes.
Checklist
bun run lintpasses (no ESLint errors)bunx tsc --noEmitpasses (no TypeScript errors)aria-label/ accessible namesconsole.logstatements left inCloses #736