Skip to content
This repository has been archived by the owner on Mar 6, 2024. It is now read-only.

refactor fetch-pollyfill #13

Merged
merged 1 commit into from
Mar 12, 2023
Merged

refactor fetch-pollyfill #13

merged 1 commit into from
Mar 12, 2023

Conversation

harjotgill
Copy link
Contributor

@harjotgill harjotgill commented Mar 12, 2023

Summary by ChatGPT

Release Notes

This pull request includes updates to imports and types from chatgpt and options.js, removes import of node-fetch, and sets global variables for fetch, Headers, Request, and Response. No new features were added, but these changes improve the organization and maintainability of the code.

@github-actions
Copy link
Contributor

🤖 ChatGPT

Thank you for letting me participate in the code review.

This pull request updates imports and types from chatgpt and options.js, removes import of node-fetch, and sets global variables for fetch, Headers, Request, and Response. The changes seem reasonable and well-organized.

Overall feedback: Good job!

Filename Summary
src/bot.ts Updates imports and types from chatgpt and options.js. Updates SendMessageOptions type to use chatgpt namespace.
src/fetch-polyfill.js Removes import of node-fetch and sets global variables for fetch, Headers, Request, and Response.
src/fetch-polyfill.ts Imports node-fetch and sets global variables for fetch, Headers, Request, and Response.

@harjotgill harjotgill changed the title test refactor fetch-pollyfill Mar 12, 2023
@harjotgill harjotgill merged commit 6dafc50 into main Mar 12, 2023
@harjotgill harjotgill deleted the test branch March 12, 2023 02:31
shoyuf pushed a commit to maybeLab/openai-mr-reviewer-gltlab that referenced this pull request Jun 25, 2023
Signed-off-by: Tao He <sighingnow@gmail.com>
shoyuf pushed a commit to maybeLab/openai-mr-reviewer-gltlab that referenced this pull request Jun 25, 2023
This pull request includes updates to imports and types from `chatgpt`
and `options.js`, removes import of `node-fetch`, and sets global
variables for `fetch`, `Headers`, `Request`, and `Response`. No new
features were added, but these changes improve the organization and
maintainability of the code.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant