Skip to content

Version 1.0.2#114

Merged
rsgalloway merged 5 commits intomasterfrom
v1.0.2
Apr 2, 2026
Merged

Version 1.0.2#114
rsgalloway merged 5 commits intomasterfrom
v1.0.2

Conversation

@rsgalloway
Copy link
Copy Markdown
Owner

@rsgalloway rsgalloway commented Apr 2, 2026

This pull request updates the project to version 1.0.2, focusing on improving the test workflow and hardening the encryption helpers. The main improvements include adding a new Makefile target for running tests in editable mode, updating the handling of optional cryptography dependencies to prevent errors, and removing problematic return statements from finally blocks to avoid Python warnings.

Test workflow improvements:

  • Added a pytest target to the Makefile that installs the package in editable mode before running the test suite, matching CI behavior.
  • Updated the .PHONY targets in the Makefile to include pytest.

Encryption helper fixes and hardening:

  • Removed return statements from finally blocks in encryption helpers to avoid newer Python SyntaxWarnings. [1] [2]
  • Improved optional cryptography imports by defining fallback classes and variables so missing crypto dependencies do not break exception handling paths. Updated exception handling to use these fallbacks. [1] [2] [3]

Version bump:

  • Bumped the project version to 1.0.2 in both pyproject.toml and lib/envstack/__init__.py. [1] [2]

Changelog update:

  • Updated CHANGELOG.md to document the new version and its changes.

@rsgalloway rsgalloway self-assigned this Apr 2, 2026
@rsgalloway rsgalloway added the bug Something isn't working label Apr 2, 2026
@rsgalloway rsgalloway added this to the Version 1.0.2 milestone Apr 2, 2026
@rsgalloway rsgalloway linked an issue Apr 2, 2026 that may be closed by this pull request
@rsgalloway rsgalloway marked this pull request as ready for review April 2, 2026 04:20
@rsgalloway rsgalloway requested a review from Copilot April 2, 2026 04:21
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR bumps EnvStack to version 1.0.2 and focuses on developer workflow improvements for running tests locally while also hardening the encryption helper module to better tolerate missing cryptography dependencies and avoid return-in-finally warnings.

Changes:

  • Bump project version to 1.0.2 in pyproject.toml and lib/envstack/__init__.py.
  • Add a make pytest target intended to mirror CI’s editable-install test flow.
  • Update lib/envstack/encrypt.py to remove return statements from finally blocks and introduce fallback symbols/classes for optional cryptography imports.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
pyproject.toml Version bump to 1.0.2.
Makefile Adds pytest target and updates .PHONY targets.
lib/envstack/encrypt.py Optional crypto import hardening; remove returns from finally; adjust InvalidTag handling.
lib/envstack/__init__.py Version bump to 1.0.2.
CHANGELOG.md Document 1.0.2 release notes.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread lib/envstack/encrypt.py
Comment thread Makefile
Comment thread Makefile Outdated
@rsgalloway rsgalloway merged commit 06ca54d into master Apr 2, 2026
36 checks passed
@rsgalloway rsgalloway deleted the v1.0.2 branch April 2, 2026 12:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

SyntaxWarning: 'return' in a 'finally' block

2 participants