Skip to content

Improve PPTX converter, fix lint/type errors, and harden release workflow - #22

Merged
hutchins merged 3 commits into
mainfrom
fix/pptx-converter-improvements
Apr 9, 2026
Merged

Improve PPTX converter, fix lint/type errors, and harden release workflow#22
hutchins merged 3 commits into
mainfrom
fix/pptx-converter-improvements

Conversation

@hutchins

@hutchins hutchins commented Apr 9, 2026

Copy link
Copy Markdown
Owner

Summary

  • PPTX converter improvements: aggregate warnings by type, add image alt text from shape name, preserve bold/italic on hyperlinks
  • Lint & type check cleanup: fix all ruff lint errors and ty type warnings across the repo, add ty.toml config, replace dict-unpacking test helpers with explicit typed constructors
  • Release workflow fix: handle pre-existing GitHub releases by uploading assets instead of failing on duplicate tags
  • Version bump: 0.8.0 → 0.8.1

Test plan

  • make tests passes clean (738 passed, 0 warnings)
  • Verify PPTX import produces aggregated warnings (e.g. "Presentation has 3 unsupported charts — skipped")
  • Verify PPTX images have alt text in markdown output
  • Verify bold hyperlinks render as **[text](url)**
  • Push a duplicate tag to verify release workflow handles it gracefully

🤖 Generated with Claude Code

hutchins added 3 commits April 9, 2026 00:33
- Aggregate PPTX warnings by type instead of per-shape for cleaner output
- Add image alt text from shape name in PPTX converter
- Preserve bold/italic formatting on hyperlinks in PPTX converter
- Add ty.toml config to handle third-party stub false positives
- Fix type annotations in source.py and test helper functions
- Fix import sorting and unused variable lint errors across test files
- Remove error-suppressing prefix from typecheck Makefile target
- Handle pre-existing GitHub releases in release workflow by uploading
  assets instead of failing on duplicate tag
- Replace **kwargs dict-unpacking helpers in test files with explicit
  typed parameters to eliminate type checker warnings
…t path

- Bump version to 0.8.1
- Suppress third-party type stub warnings in ty.toml instead of demoting
- Update CIButler integration test path
@hutchins
hutchins merged commit 8f1d699 into main Apr 9, 2026
1 check 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.

1 participant