Skip to content

refactor(types): declare resume shapes as interfaces - #34

Merged
helderberto merged 1 commit into
mainfrom
refactor/structure-8-interfaces
Oct 24, 2026
Merged

helderberto merged 1 commit into
mainfrom
refactor/structure-8-interfaces

Conversation

@helderberto

Copy link
Copy Markdown
Owner

Context

Part 8, on top of #33. TypeScript rule (helderberto/dotfiles#14): interface for object shapes, type only for unions, tuples, mapped/conditional types and typeof derivations.

Summary

Position, Experience and SkillGroup in src/types/resume.ts switch from type to interface. They were the only object type aliases in src; there are no enums, any or eslint-disable comments.

Evidence

  • Before / After: types only, no emitted code changes. tsc, ESLint, 205 tests and next build pass.

Merge danger

Door: two-way

Blast radius: resume


Generated by Claude Code

@vercel

vercel Bot commented Oct 24, 2026 •

Copy link
Copy Markdown

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

Project Deployment Actions Updated
website Ready Ready Preview Oct 24, 2026 10:57am UTC

@helderberto
helderberto force-pushed the refactor/project-structure branch from d4babc4 to d4aaa47 Compare October 24, 2026 10:55
Base automatically changed from refactor/project-structure to main October 24, 2026 10:55
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01DZqAJbwwPeQSQFaPLDDSfr
@helderberto
helderberto force-pushed the refactor/structure-8-interfaces branch from a99c77e to 042cbd0 Compare October 24, 2026 10:55
@helderberto
helderberto merged commit 5eec029 into main Oct 24, 2026
1 of 2 checks passed
@helderberto
helderberto deleted the refactor/structure-8-interfaces branch October 24, 2026 10:55

This branch was successfully deployed

1 active deployment
Preview — 042cbd02 Deployed Oct 24, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants