Skip to content

Handle invalid versions in valid()#54

Open
StantonMatt wants to merge 1 commit into
podhmo:masterfrom
StantonMatt:codex/valid-invalid-version-none
Open

Handle invalid versions in valid()#54
StantonMatt wants to merge 1 commit into
podhmo:masterfrom
StantonMatt:codex/valid-invalid-version-none

Conversation

@StantonMatt
Copy link
Copy Markdown

@StantonMatt StantonMatt commented Jun 1, 2026

Summary

  • return None from valid() when parse() cannot build a semantic version
  • add regression coverage for invalid version strings in strict and loose modes

Fixes #42.

Tests

  • venv/bin/python -m pytest nodesemver/tests/test_regression.py -q
  • venv/bin/python -m pytest -q

Review

  • review-fix-loop clean
  • fresh Codex review found no blocking issues

@StantonMatt StantonMatt marked this pull request as ready for review June 1, 2026 18:53
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.

Unhandled AttributeError in parse() call

1 participant