Skip to content

Moved to new Diff endpoint and fix with commenting logic #88

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 5 commits into from
Jun 24, 2025

Conversation

dacoburn
Copy link
Collaborator

Root Cause

Fixed issue with commenting an overview comment on removed packages

Fix

Removed the check for removed packages in the overview logic

Public Changelog

N/A

@dacoburn dacoburn requested a review from a team as a code owner June 24, 2025 05:32
@dacoburn dacoburn requested review from alxhotel and jfblaa and removed request for a team June 24, 2025 05:32
Copy link

github-actions bot commented Jun 24, 2025

🚀 Preview package published!

Install with:

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

Docker image: socketdev/cli:pr-88

@dacoburn dacoburn changed the title Version bump Moved to new Diff endpoint and fix with commenting logic Jun 24, 2025
Copy link

@rchatrath7 rchatrath7 left a comment

Choose a reason for hiding this comment

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

seems okay

@dacoburn dacoburn merged commit 82d300b into main Jun 24, 2025
6 checks passed
@dacoburn dacoburn deleted the doug/add-new-diff-endpoint-logic branch June 24, 2025 05: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.

2 participants