Skip to content

Improve code maintainability and test quality#41

Merged
Hiddentale merged 14 commits into
mainfrom
improve_maintainability
Apr 3, 2026
Merged

Improve code maintainability and test quality#41
Hiddentale merged 14 commits into
mainfrom
improve_maintainability

Conversation

@Hiddentale

Copy link
Copy Markdown
Owner

Description

Refactor the codebase for better maintainability: extract magic constants, simplify functions, improve test organization, and add a VkError wrapper.

Key changes:

  • Replace magic numbers with named constants and a version struct
  • Extract classify_params into focused sub-functions
  • Extract generate_layout_checks and generate_wrappers from main
  • Consolidate duplicated tests and extract shared test helpers into a common module
  • Improve test names to describe scenario and expected outcome
  • Add VkError wrapper and fail on rustfmt errors
  • Update contributing docs and README

Type of Change

  • Bug fix
  • Documentation update

Checklist

  • I ran cargo fmt --all (code is formatted)
  • I ran cargo clippy --workspace --all-targets -- -D warnings (no warnings)
  • I ran cargo test --workspace (all tests pass)
  • I added tests for new functionality
  • I updated relevant documentation

@Hiddentale Hiddentale self-assigned this Apr 3, 2026
@Hiddentale Hiddentale merged commit c241750 into main Apr 3, 2026
17 checks passed
@Hiddentale Hiddentale deleted the improve_maintainability branch April 3, 2026 10:49
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.

1 participant