Skip to content

release: cleanup dead scripts and tests#20

Merged
gdtknight merged 1 commit into
mainfrom
develop
Apr 17, 2026
Merged

release: cleanup dead scripts and tests#20
gdtknight merged 1 commit into
mainfrom
develop

Conversation

@gdtknight
Copy link
Copy Markdown
Owner

Description

developmain으로 병합합니다. 이번 릴리즈 주기에는 PR #19(유지보수 아티팩트 정리)만 포함되어 있어 런타임 동작이나 공개 API에는 변경이 없습니다.

Related Issue

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update
  • Code refactoring
  • Performance improvement
  • Build/CI improvement

Changes

Testing

Test Environment

  • OS: Ubuntu (Linux 6.8.0-106-generic), macOS (CI)
  • Compiler: cc

Test Steps

  1. PR chore: remove dead scripts, tests, and stale gitignore entries #19 머지 전 15개 CI 체크 전부 통과 확인 (Linux/macOS 빌드, norminette, memory leak detection)
  2. make fclean && make all — 로컬 재빌드 성공
  3. ./miniRT scenes/valid/valid_smoke_simple.rt — 시각 검증
  4. git log origin/main..origin/develop — 포함 커밋 단일 확인

Test Results

  • All existing tests pass
  • New tests added and passing
  • Manual testing completed
  • Norminette check passed
  • No memory leaks (valgrind/leaks)

Screenshots

Checklist

  • Code follows project style guidelines (norminette)
  • Self-review of code completed
  • Comments added for complex logic
  • Documentation updated (if needed)
  • No new warnings generated
  • Tests added for new functionality
  • All tests passing locally
  • Branch is up to date with base branch
  • Commit messages follow conventional format
  • No sensitive data included

Constitution Compliance

  • 42 Norminette compliance verified
  • Code readability prioritized
  • Unit tests created and passing
  • Documentation updated (English + Korean if applicable)
  • Build verification completed
  • Git branch strategy followed
  • Logs stored in logs/ directory

Performance Impact

  • No performance impact
  • Performance improved
  • Performance decreased (justified below)

Additional Notes

  • develop → main은 브랜치 전략상 merge commit 방식으로 병합해야 합니다(스쿼시 아님 — 통합 시점 보존 목적). gh CLI로 머지할 때 --merge 플래그를 사용하세요.
  • 이번 릴리즈는 유지보수성 개선만 포함하므로 태그가 필요하다면 PATCH(v2.8.1)가 적절합니다.

🤖 Generated with Claude Code

Clean up maintenance artifacts that have outlived their purpose:

- tests/: test_shadow_calc.c no longer links (calculate_shadow_bias
  is now static); test_shadow_config binary is an orphan with no
  source; baselines/metadata.json captured pre-refactor snapshot
  that is no longer relevant
- scripts/: remove one-off validators tied to archived specs
  (012 cleanup, 013 norminette, BVH integration fix), superseded
  helpers (create_wiki.sh replaced by .github/scripts/sync-wiki.sh),
  norm-violating tooling (fix_whitespace.sh converted tabs to spaces),
  and scripts with broken references (fix_line_continuations.sh)
- scripts/test/: all three scripts dead (hardcoded macOS path in
  test_progress.sh, outdated spec 002 validation)
- .gitignore: drop patterns for test binaries that never existed
  (test_vector, test_parser, test_mlx, etc.) and the removed
  test_optimizations.sh script

Remaining in scripts/: validate_norminette.sh, test_miniRT.sh,
test_scenes.sh, bench.sh — all referenced in AGENTS.md and wiki.

Co-authored-by: Claude <noreply@anthropic.com>
@gdtknight gdtknight self-assigned this Apr 16, 2026
@gdtknight gdtknight moved this from Backlog to In review in @gdtknight's miniRT Apr 16, 2026
@gdtknight gdtknight merged commit 7c1f333 into main Apr 17, 2026
16 checks passed
@github-project-automation github-project-automation Bot moved this from In review to Done in @gdtknight's miniRT Apr 17, 2026
gdtknight added a commit that referenced this pull request Apr 17, 2026
README was stuck at v2.6.0 and docs/CHANGELOG.md at v2.4.0, both
missing the accumulated release history from the package
restructuring work.

- README: rewrite the project structure tree to reflect the
  7-package feature-based layout introduced in v2.8.0 (common,
  scene, spatial, render, lighting, interact, metrics), replacing
  the stale flat layout. Extend the release history table with
  entries for v2.4.1, v2.5.0, v2.5.1, v2.6.0, v2.6.1, v2.6.2,
  v2.7.0, v2.8.0, and v2.8.1.
- CHANGELOG: add full Keep-a-Changelog entries for v2.4.1
  through v2.8.1, including the module restructurings, header
  cleanups, camera pitch fix, and the v2.8.1 maintenance
  cleanup (PRs #19, #20).

Pure documentation update — no source or build changes.

Co-Authored-By: Claude <noreply@anthropic.com>
gdtknight added a commit that referenced this pull request Apr 17, 2026
README was stuck at v2.6.0 and docs/CHANGELOG.md at v2.4.0, both
missing the accumulated release history from the package
restructuring work.

- README: rewrite the project structure tree to reflect the
  7-package feature-based layout introduced in v2.8.0 (common,
  scene, spatial, render, lighting, interact, metrics), replacing
  the stale flat layout. Extend the release history table with
  entries for v2.4.1, v2.5.0, v2.5.1, v2.6.0, v2.6.1, v2.6.2,
  v2.7.0, v2.8.0, and v2.8.1.
- CHANGELOG: add full Keep-a-Changelog entries for v2.4.1
  through v2.8.1, including the module restructurings, header
  cleanups, camera pitch fix, and the v2.8.1 maintenance
  cleanup (PRs #19, #20).

Pure documentation update — no source or build changes.

Co-authored-by: Claude <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant