Skip to content

Conversation

@Artawower
Copy link
Contributor

No description provided.

Copy link
Member

@jcs090218 jcs090218 left a comment

Choose a reason for hiding this comment

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

I wonder how to use this patch? If you add (add-to-list 'codegpt-action-alist '("JSDoc" . "Write JSDoc for this method")), would it work?

Thanks for working on this! ❤️ 👍

@Artawower
Copy link
Contributor Author

I wonder how to use this patch? If you add (add-to-list 'codegpt-action-alist '("JSDoc" . "Write JSDoc for this method")), would it work?

Sure!

Screen.Recording.2023-02-05.at.03.04.19.mov

@Artawower
Copy link
Contributor Author

BTW, do you think it is possible to format the provided code from openai? Because now it looks like a solid line without line breaks

@jcs090218
Copy link
Member

Sure!

👍

BTW, do you think it is possible to format the provided code from openai? Because now it looks like a solid line without line breaks

I think this is caused by

(fill-region original-point (point))))

We will need to by pass that operation to make the code remain untouch.

@Artawower Artawower force-pushed the custom-predefined-actions branch from bd3c9bd to 4b9a5eb Compare February 4, 2023 23:53
@Artawower
Copy link
Contributor Author

Corrected the call of standard functions, there was an error with the extraction of the standard function due to unnecessary car-safe

@jcs090218 jcs090218 merged commit 850eea1 into emacs-openai:master Feb 5, 2023
@jcs090218
Copy link
Member

Code LGTM! Thanks!

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