Skip to content

Conversation

Zrt781
Copy link

@Zrt781 Zrt781 commented Apr 3, 2023

.env.example

@mingyi77
Copy link

mingyi77 commented Apr 9, 2023

/gpt hello

@anc95
Copy link
Owner

anc95 commented May 26, 2023

the api key is expired, I just updated, please push new commit to test. enjoy

@ArnavK-09
Copy link

/gpt hello

Copy link

@bhattk64 bhattk64 left a comment

Choose a reason for hiding this comment

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

Hi @Feng-ATM,

Thanks for the PR! I see this is a large refactor with a lot of changes, including:

  • Updates to the .env.example file with new variables.
  • Addition of a new WebAssembly binary file (_tiktoken_bg.wasm).
  • Major rewrite of action/index.cjs with tens of thousands of additions and deletions.

Could you please help by adding a summary to the PR description explaining:

  • The goals of this refactor/change.
  • Why the .wasm file was added and its purpose.
  • Any breaking changes or things maintainers/reviewers should focus on.

Also, it would be helpful to have some inline comments in the code for key parts of the new logic, to make reviewing easier.

Looking forward to your clarifications before diving deeper into the code!

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.

5 participants