Skip to content

Add comprehensive test suite and CI/CD setup#1

Merged
trissim merged 1 commit intomasterfrom
claude/add-tests-and-docs-011CUg8DhgUzShix8mcwpK3k
Nov 1, 2025
Merged

Add comprehensive test suite and CI/CD setup#1
trissim merged 1 commit intomasterfrom
claude/add-tests-and-docs-011CUg8DhgUzShix8mcwpK3k

Conversation

@trissim
Copy link
Owner

@trissim trissim commented Oct 31, 2025

  • Add 69 comprehensive tests covering all major functionality

    • SignatureAnalyzer tests for functions, methods, dataclasses
    • UnifiedParameterAnalyzer tests for unified API
    • DocstringExtractor tests for various docstring formats
    • Exception hierarchy tests
    • Public API and import tests
  • Achieve 65% overall test coverage (92% for UnifiedParameterAnalyzer)

  • Enhance GitHub Actions CI workflow

    • Add pip caching for faster builds
    • Separate lint job from test job
    • Add build verification job
    • Update to latest action versions (v4/v5)
    • Add support for develop branch
    • Set continue-on-error for linting to not block CI
  • Update publish workflow

    • Add workflow_dispatch for manual triggers
    • Update to latest action versions
    • Add permissions for secure publishing
  • Add comprehensive CONTRIBUTING.md guide

    • Development setup instructions
    • Testing guidelines
    • Code style requirements
    • Pull request process
    • Bug reporting template
  • Update README.md

    • Add CI and PyPI badges
    • Add development section with setup instructions
    • Add testing instructions
  • Fix bugs in source code

    • Fix DocstringInfo to always return dict for parameters (never None)
    • Fix parameter skipping logic to not skip regular function params
    • Add dunder field filtering for dataclasses
    • Handle callable objects properly in UnifiedParameterAnalyzer
    • Fix docstring_info.parameters access when docstring_info is None

- Add 69 comprehensive tests covering all major functionality
  - SignatureAnalyzer tests for functions, methods, dataclasses
  - UnifiedParameterAnalyzer tests for unified API
  - DocstringExtractor tests for various docstring formats
  - Exception hierarchy tests
  - Public API and import tests
- Achieve 65% overall test coverage (92% for UnifiedParameterAnalyzer)

- Enhance GitHub Actions CI workflow
  - Add pip caching for faster builds
  - Separate lint job from test job
  - Add build verification job
  - Update to latest action versions (v4/v5)
  - Add support for develop branch
  - Set continue-on-error for linting to not block CI

- Update publish workflow
  - Add workflow_dispatch for manual triggers
  - Update to latest action versions
  - Add permissions for secure publishing

- Add comprehensive CONTRIBUTING.md guide
  - Development setup instructions
  - Testing guidelines
  - Code style requirements
  - Pull request process
  - Bug reporting template

- Update README.md
  - Add CI and PyPI badges
  - Add development section with setup instructions
  - Add testing instructions

- Fix bugs in source code
  - Fix DocstringInfo to always return dict for parameters (never None)
  - Fix parameter skipping logic to not skip regular function params
  - Add dunder field filtering for dataclasses
  - Handle callable objects properly in UnifiedParameterAnalyzer
  - Fix docstring_info.parameters access when docstring_info is None
@trissim trissim merged commit bf71e5d into master Nov 1, 2025
14 checks passed
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