Skip to content

refactor(remotion): name demo animation constants - #50

Merged
eduardbar merged 4 commits into
masterfrom
refactor/remotion-readme-constants
Aug 3, 2026
Merged

refactor(remotion): name demo animation constants#50
eduardbar merged 4 commits into
masterfrom
refactor/remotion-readme-constants

Conversation

@eduardbar

Copy link
Copy Markdown
Owner

Refs #42

Summary

  • Name DriftReadmeDemo animation frames, dimensions, offsets, scales, and layout values with semantic constants.
  • Preserve composition registration, props, exact timings, colors, and rendered behavior.
  • Reduce the target's self-scan findings from 23 to 2 without rule/config changes.

Verification

  • Remotion TypeScript check
  • Root build and full tests: 351 passed
  • Frame 72 render sanity verified at 1200x675
  • git diff --check
  • Scoped diff: 56 changed lines

Follow-up

Component extraction and stage-transition coverage are intentionally deferred to a separate Slice B.

@vercel

vercel Bot commented Aug 3, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
site Ready Ready Preview Aug 3, 2026 11:15pm

@github-actions

github-actions Bot commented Aug 3, 2026

Copy link
Copy Markdown

drift trust

drift trust

  • Trust Score: 73/100
  • Merge Risk: MEDIUM
  • Target: /home/runner/work/drift/drift

Diff signals

  • Base ref: origin/master
  • Diff status: IMPROVED
  • Score delta: +0
  • Issues: +1 new / -22 resolved
  • Trust adjustment: +3 penalty / -20 bonus (net -17)

Top reasons

  • Error-level issues: 8 error issue(s) increase merge volatility. (impact 22)
  • Highest-risk file: /home/runner/work/drift/drift/site/src/App.jsx has hotspot risk 70/100. (impact 11)
  • Maintenance hotspots: Maintenance risk is MEDIUM (36/100). (impact 9)
  • Drift score pressure: Repository drift score is 4/100. (impact 2)

Fix priorities

drift review

drift review

⚠️ Drift regressed: +0 score and 1 new issue(s).

  • Base ref: origin/master
  • Score delta: +0
  • New issues: 1
  • Resolved issues: 22

File breakdown

  • /home/runner/work/drift/drift/remotion/compositions/DriftReadmeDemo.tsx: 98 -> 35 (-63), +1 new / -22 resolved

@eduardbar
eduardbar merged commit 1c399f7 into master Aug 3, 2026
7 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type:quality Quality gates and testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant