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.

Sign up for GitHub

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

Jump to bottom

Fixing lint errors in concepts including hooks #1088

Open
wants to merge 11 commits into
base: main
Choose a base branch
from
Open

Fixing lint errors in concepts including hooks #1088

wants to merge 11 commits into from

Commits on Oct 17, 2024

  1. fix: added off and clear attributes to Instructor base class

    @ivanleomk
    ivanleomk committed Oct 17, 2024
    Configuration menu
    Copy the full SHA
    eeb7e1d View commit details
    Browse the repository at this point in the history

  2. fix: added tests to ensure hooks are working as intended

    @ivanleomk
    ivanleomk committed Oct 17, 2024
    Configuration menu
    Copy the full SHA
    8ff960a View commit details
    Browse the repository at this point in the history

  3. fix: removed duplicate on definition in instructor

    @ivanleomk
    ivanleomk committed Oct 17, 2024
    Configuration menu
    Copy the full SHA
    09ab14e View commit details
    Browse the repository at this point in the history

  4. fix: removed unused import in test_hooks

    @ivanleomk
    ivanleomk committed Oct 17, 2024
    Configuration menu
    Copy the full SHA
    218070d View commit details
    Browse the repository at this point in the history

  5. fix: remove commented line test

    @ivanleomk
    ivanleomk committed Oct 17, 2024
    Configuration menu
    Copy the full SHA
    50b3d55 View commit details
    Browse the repository at this point in the history

  6. fix: added more tests

    @ivanleomk
    ivanleomk committed Oct 17, 2024
    Configuration menu
    Copy the full SHA
    f4d89cc View commit details
    Browse the repository at this point in the history

  7. fix: added more tests for hooks

    @ivanleomk
    ivanleomk committed Oct 17, 2024
    Configuration menu
    Copy the full SHA
    b5c3e03 View commit details
    Browse the repository at this point in the history

  8. fix: Added None type default argument for client

    @ivanleomk
    ivanleomk committed Oct 17, 2024
    Configuration menu
    Copy the full SHA
    30e169c View commit details
    Browse the repository at this point in the history

  9. docs:fix hooks.md

    @ivanleomk
    ivanleomk committed Oct 17, 2024
    Configuration menu
    Copy the full SHA
    39b90cc View commit details
    Browse the repository at this point in the history

  10. docs: fix multimodal

    @ivanleomk
    ivanleomk committed Oct 17, 2024
    Configuration menu
    Copy the full SHA
    8a3f932 View commit details
    Browse the repository at this point in the history

  11. docs: Added examples of hook outputs

    @ivanleomk
    ivanleomk committed Oct 17, 2024
    Configuration menu
    Copy the full SHA
    0bd7b30 View commit details
    Browse the repository at this point in the history