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

UI Type checks for build #378

Merged
merged 3 commits into from
May 30, 2024
Merged

UI Type checks for build #378

merged 3 commits into from
May 30, 2024

Conversation

nerdalert
Copy link
Member

@nerdalert nerdalert commented May 29, 2024

  • Fix type errors for prod building (npm run build)
  • Add type checking to CI
  • Changed custom endpoints to be processed on the client side. Most of the time that would be a user inferring on localhost or an endpoint on their private network.
  • Added a cleanup button for playground chat.

- Fix type errors for prod building (npm run build)
- Add type checking to CI

Signed-off-by: Brent Salisbury <[email protected]>
@nerdalert nerdalert marked this pull request as draft May 29, 2024 05:31
@nerdalert
Copy link
Member Author

nerdalert commented May 29, 2024

@vishnoianil double check knowledge/route submission at your leisure. Had to change them to pass type checking to get prod built. Ty!

@nerdalert nerdalert marked this pull request as ready for review May 29, 2024 14:49
nerdalert added 2 commits May 29, 2024 20:47
- Custom endpoints should be processed from the client
side since it will often be a localhost inference.
- We also wouldn't want arbitrary proxying to models from
the server side other than the predefined endpoints provided.

Signed-off-by: Brent Salisbury <[email protected]>
@vishnoianil
Copy link
Member

@vishnoianil double check knowledge/route submission at your leisure. Had to change them to pass type checking to get prod built. Ty!

@nerdalert tested skill/knowledge both, looks good.

@mergify mergify bot merged commit b5ac4a5 into instructlab:main May 30, 2024
10 checks passed
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