Skip to content

v3.1.1: Fix snapshot crash, PyPI naming, and CI pipeline#744

Merged
farchide merged 1 commit intomasterfrom
release/v3.1.0
Mar 4, 2026
Merged

v3.1.1: Fix snapshot crash, PyPI naming, and CI pipeline#744
farchide merged 1 commit intomasterfrom
release/v3.1.0

Conversation

@farchide
Copy link
Collaborator

@farchide farchide commented Mar 4, 2026

Summary

  • Fix eliminate_duplicate_snapshots crash: Added isinstance(value, list) guards to prevent TypeError: 'bool' object is not iterable when snapshot_data contains non-list values (e.g., booleans)
  • Fix PyPI upload: Changed package name from prancer-basic to prancer_basic in setup.py to produce valid sdist filenames
  • Fix CI pipeline: Changed Docker test invocation from sh to bash (fixes [[: not found errors), added OPA binary installation in dev-test.sh
  • Fix terraform parser: Replaced deprecated inspect.getargspec with inspect.getfullargspec for Python 3.12 compatibility
  • Fix ternary expression evaluation: Added proper comparison operator handling (<, >, ==, etc.) in conditional expressions instead of falling back to bool(string)
  • Version bump: 3.1.0 → 3.1.1

Test plan

  • Full test suite: 991 passed, 0 failed
  • Verify PyPI upload produces prancer_basic-3.1.1.tar.gz
  • Verify CI pipeline runs with OPA available in Docker containers
  • Verify AWS snapshot crawling with non-list snapshot_data entries

🤖 Generated with Claude Code

- Fix TypeError in eliminate_duplicate_snapshots when snapshot_data contains
  non-list values (e.g., booleans) by adding isinstance guards
- Change package name from prancer-basic to prancer_basic for valid PyPI uploads
- Fix CI pipeline: use bash instead of sh for [[ operators, install OPA binary
- Bump version to 3.1.1

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@farchide farchide requested a review from a team March 4, 2026 06:56
@farchide farchide merged commit a03c78c into master Mar 4, 2026
3 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.

1 participant