Skip to content

Conversation

@dacoburn
Copy link
Collaborator

@dacoburn dacoburn commented Apr 3, 2025

When no top level ancestors the cli would fail

Root Cause

assuming an attribute existed when it did not or was none

Fix

Added a check for the attribute

Public Changelog

  • Fix for a case where an attribute of a package could be None and cause the cli to fail

@dacoburn dacoburn added the Product Changelog New features for the public changelog label Apr 3, 2025
@dacoburn dacoburn requested a review from a team as a code owner April 3, 2025 18:46
@dacoburn dacoburn requested review from bcomnes and bmeck and removed request for a team April 3, 2025 18:46
@github-actions
Copy link

github-actions bot commented Apr 3, 2025

🚀 Preview package published!

Install with:

pip install --index-url https://test.pypi.org/simple/ --extra-index-url https://pypi.org/simple socketsecurity==2.0.36.dev1

Docker image: socketdev/cli:pr-68

@obarrera obarrera self-requested a review April 3, 2025 18:47
@dacoburn dacoburn merged commit 7198230 into main Apr 3, 2025
6 checks passed
@dacoburn dacoburn deleted the doug/fix-failed-fulled-scan branch April 3, 2025 18:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Product Changelog New features for the public changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants