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

Add description and other examples #29

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Add description and other examples #29

wants to merge 1 commit into from

Conversation

jerhadf
Copy link
Contributor

@jerhadf jerhadf commented Apr 24, 2024

No description provided.

| [`evi-typescript-example`](/evi-typescript-example/README.md) | Typescript | Vanilla JS | |
| [`evi-custom-language-model`](/evi-custom-language-model/README.md) | Python | | |
| [`meld`](/meld/README.md) | Typescript | React | |
| [`next-app`](https://github.com/HumeAI/empathic-voice-api-js/tree/main/examples/next-app) | Typescript | Next.js | |
Copy link
Member

Choose a reason for hiding this comment

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

@jerhadf
These are shell apps within the empathic-voice-api-js monorepo. (The purpose of these shell apps are to test packages published from the monorepo locally)

There is some work to be done to make these into example apps which serve as external developer resources:

  1. Update project names to reflect what they are an example of.
  2. Update project READMEs to include: descriptions of what the apps are and what they do, env setup instructions, instructions to build/run the projects, and instructions for usage.
  3. Comments throughout the code which act as documentation for the code, explaining various steps is also really good to have— but can be added later based on developer feedback (like when something is called out as unclear)

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