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

Support for the V ORM #102

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

Support for the V ORM #102

wants to merge 1 commit into from

Commits on Dec 31, 2023

  1. Support for the V ORM

    DO NOT MERGE.
    
    This is partial support for the ORM. However, there are some challenges
    that need to be addressed before this can be properly reviewed and
    landed:
    
    1. The ORM in V requires drivers to be hard-coded. See #90.
    
    2. The Connection doesn't _really_ implement orm.Connection because the
    vsql Connection is required to be mut and the current interface
    definition does not allow this.
    
    3. We need to create a new test suite for the ORM. `vsql/orm_test.v`
    filled with combinations of statements "sql" commands will work just
    fine. Specifically, we need to test different combinations of
    expressions and types.
    
    Fixes #90
    elliotchance committed Dec 31, 2023
    Configuration menu
    Copy the full SHA
    dc6a494 View commit details
    Browse the repository at this point in the history