Skip to content
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

PATCHED_API_KEY integration #12

Merged
merged 8 commits into from
Apr 12, 2024
Merged

PATCHED_API_KEY integration #12

merged 8 commits into from
Apr 12, 2024

Conversation

CTY-git
Copy link
Contributor

@CTY-git CTY-git commented Apr 9, 2024

No description provided.

Copy link
Member

@codelion codelion left a comment

Choose a reason for hiding this comment

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

LGTM!

@patched-admin
Copy link
Contributor

patched-admin commented Apr 12, 2024

The code changes involve importing the traceback module, adding exception handling, and printing traceback information and error messages in case of an exception. Additionally, modifications to the 'create_comment' method in the 'scm.py' file of the patchwork module include setting a 'note' based on 'path,' handling file differences, and changing the logic for extracting 'slug' from a PR URL. In the 'CallOpenAI.py' file, the required_keys attribute in the CallOpenAI class is updated, constants for _TOKEN_URL and _DEFAULT_PATCH_URL are introduced, and input API key handling is modified with validation checks. A new function transform_sarif_results replaces existing code in the ExtractCode class, generating code context based on parameters. Other changes involve formatting in the ModifyCode.py file for better readability and adding filter_by_extension function in the ReadPRDiffs class to ignore specific file extensions during processing.


@CTY-git CTY-git merged commit 731cdad into main Apr 12, 2024
1 check passed
@CTY-git CTY-git deleted the patched-api-key-integration branch April 12, 2024 10:54
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.

3 participants