Skip to content
This repository was archived by the owner on Jun 5, 2025. It is now read-only.

Warn about malicious, deprecated or archived packages in output snippets #215

Merged
merged 2 commits into from
Dec 6, 2024
Merged

Warn about malicious, deprecated or archived packages in output snippets #215

merged 2 commits into from
Dec 6, 2024

Conversation

jhrozek
Copy link
Contributor

@jhrozek jhrozek commented Dec 5, 2024

  • Add comments on archived, malicious or deprecated packages in snippets - Uses the storage engine to find information on packages parsed from the code snippet created by LLM. In addition to printing all the packages, if any package is marked as deprecated, archived or malicious, it is also warned about loudly.
  • Do not import src.codegate, just codegate - caught during a review earlier today

@jhrozek
Add comments on archived, malicious or deprecated packages in snippets
023fd63 
Uses the storage engine to find information on packages parsed from the
code snippet created by LLM. In addition to printing all the packages,
if any package is marked as deprecated, archived or malicious, it is
also warned about loudly.
@jhrozek
Do not import src.codegate, just codegate
e61d782
@jhrozek jhrozek marked this pull request as ready for review December 5, 2024 21:53
ptelang
ptelang approved these changes Dec 6, 2024
View reviewed changes
Copy link
Contributor

@ptelang ptelang left a comment

Choose a reason for hiding this comment

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

Looks good!

@aponcedeleonch aponcedeleonch merged commit 0e586db into stacklok:main Dec 6, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@ptelang ptelang ptelang approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@jhrozek @ptelang @aponcedeleonch