jcrodriguez1989 / chatgpt

Interface to ChatGPT from R
GNU General Public License v3.0
312 stars 37 forks source link

Release v0.2.4 #60

Open jcrodriguez1989 opened 1 month ago

github-actions[bot] commented 1 month ago

I have reviewed the GitHub PR diff provided:

Issues Found:

  1. In R/ask_chatgpt.R file, the parameter session_id and openai_api_key were removed from the ask_chatgpt function without updating the function documentation accordingly.

    • File: R/ask_chatgpt.R
    • Lines: 8-10
  2. In R/ask_chatgpt.R file, the function reset_chat_session is called without the session_id parameter, which was removed from the function signature.

    • File: R/ask_chatgpt.R
    • Line: 19
  3. In R/reset_chat_session.R file, the system_role parameter in the reset_chat_session function is not actually used within the function.

    • File: R/reset_chat_session.R
    • Line: 10

Note:

Let me know if you need any further assistance.

github-actions[bot] commented 1 month ago

I have reviewed the GitHub PR diff and found the following issues:

In the file R/ask_chatgpt.R, on line 6, the parameter session_id and openai_api_key have been removed from the ask_chatgpt function definition without updating the associated documentation. It seems like these parameters are no longer used in the function, so the documentation should be updated accordingly or removed.

In the file R/chatgpt-package.R, on line 6, the assignment of chat_session_messages on startup seems to be changed to include a specific message. This change might unexpectedly overwrite any initial values set for chat_session_messages.

Those are the issues I identified during the code review.