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

simple insert_dict() builder #725

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

simple insert_dict() builder #725

wants to merge 4 commits into from

Commits on Feb 26, 2024

  1. simple insert_dict() builder

    @edvardm
    edvardm committed Feb 26, 2024
    Configuration menu
    Copy the full SHA
    8822c88 View commit details
    Browse the repository at this point in the history

  2. CR: just use class method (Query().into -> Query.into)

    @edvardm
    edvardm committed Feb 26, 2024
    Configuration menu
    Copy the full SHA
    1d8b141 View commit details
    Browse the repository at this point in the history

  3. make insert_dict() api less confusing 

    in SQL builders, order of calling individual functions should
not matter. Originally it was allowed to first call columns()
and then insert_dict as long as the two matched, but I'm not even sure
it worked, as we should have used set comparison instead.

It is also better to have symmetric API in
    @edvardm
    edvardm committed Feb 26, 2024
    Configuration menu
    Copy the full SHA
    3167376 View commit details
    Browse the repository at this point in the history

Commits on Mar 11, 2024

  1. insert_into() example to README

    @edvardm
    edvardm committed Mar 11, 2024
    Configuration menu
    Copy the full SHA
    98cbf46 View commit details
    Browse the repository at this point in the history