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

Staging/deploy #639

Closed
wants to merge 12 commits into from
Closed

Staging/deploy #639

wants to merge 12 commits into from

Conversation

BeatrixCohere
Copy link
Collaborator

@BeatrixCohere BeatrixCohere commented Aug 12, 2024

Thank you for contributing to the Cohere Toolkit!

  • PR title: "area: description"

    • Where "area" is whichever of interface, frontend, model, tools, backend, etc. is being modified. Use "docs: ..." for purely docs changes, "infra: ..." for CI changes.
    • Example: "deployment: add Azure model option"
  • PR message: Delete this entire checklist and replace with

    • Description: a description of the change
    • Issue: the issue # it fixes, if applicable
    • Dependencies: any dependencies required for this change
  • Add tests and docs: Please include testing and documentation for your changes

  • Lint and test: Run make lint and make run-tests

AI Description

This PR makes changes to multiple files, primarily involving updates to code structure, dependencies, and configuration settings.

Code Changes:

  • src/backend/config/auth.py: Enabled OpenIDConnect as the default authentication strategy.
  • src/backend/main.py: Removed the verify_migrate_token import and updated the /migrate endpoint to use Alembic migrations directly. Added error logging for migration failures.
  • src/backend/model_deployments/cohere_platform.py: Updated error logging to include the HTTP response text when retrieving models fails.
  • src/backend/tools/google_drive/tool.py: Modified the is_available method to check for CLIENT_SECRET instead of CLIENT_ID.
  • src/interfaces/assistants_web/.env.{development,production}: Updated the NEXT_PUBLIC_FRONTEND_HOSTNAME to use port 4000 instead of 4444.
  • src/interfaces/assistants_web/package.json: Changed the port used in the "dev", "start", and "start:single-docker" scripts from 4444 to 4000.
  • src/interfaces/assistants_web/src/constants.ts: Added a new field is_private to the BASE_AGENT constant, setting it to false.
  • src/backend/routers/tool.py: Added an import for Settings and included a log statement to display the Redis URL.
  • .github/workflows/docker_push_frontend.yml: Modified the context path from ./src/interfaces/coral_web to ./src/interfaces/assistants_web.

@codecov-commenter
Copy link

codecov-commenter commented Aug 15, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 75.23%. Comparing base (ca5ab66) to head (56c5909).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #639   +/-   ##
=======================================
  Coverage   75.23%   75.23%           
=======================================
  Files         265      265           
  Lines       11862    11862           
=======================================
  Hits         8924     8924           
  Misses       2938     2938           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

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