Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

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
dacoburn merged 5 commits into main from doug/add-new-diff-endpoint-logic
Jun 24, 2025

Conversation

@dacoburn
Copy link
Collaborator

@dacoburn dacoburn commented Jun 24, 2025

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
edited
Loading

🚀 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

Reviewers

@rchatrath7 rchatrath7 rchatrath7 approved these changes

@alxhotel alxhotel Awaiting requested review from alxhotel alxhotel is a code owner automatically assigned from SocketDev/eng

@jfblaa jfblaa Awaiting requested review from jfblaa jfblaa is a code owner automatically assigned from SocketDev/eng

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /