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

Better check for whether or not a table is_chrono? #98

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

Better check for whether or not a table is_chrono? #98

wants to merge 6 commits into from

Commits on Sep 12, 2022

  1. Better check for whether or not a table is_chrono? 

    The previous check didn't work properly because `data_source_sql`
(which `data_source_exists?` uses internally) only qualifies the table_name
to the current schema if a fully qualified name isn't passed.
    @pnomolos
    pnomolos committed Sep 12, 2022
    Configuration menu
    Copy the full SHA
    37c5c45 View commit details
    Browse the repository at this point in the history

  2. Make sure to check table directly in case schema is passed in

    @pnomolos
    pnomolos committed Sep 12, 2022
    Configuration menu
    Copy the full SHA
    e1dd13b View commit details
    Browse the repository at this point in the history

  3. Added specs

    @pnomolos
    pnomolos committed Sep 12, 2022
    Configuration menu
    Copy the full SHA
    85bb457 View commit details
    Browse the repository at this point in the history

  4. Fix spec so table is reset after the spec is run.

    @pnomolos
    pnomolos committed Sep 12, 2022
    Configuration menu
    Copy the full SHA
    19905b5 View commit details
    Browse the repository at this point in the history

Commits on Sep 16, 2022

  1. Create spec for a table name with a period in it, and fixed issues ar… 

    …ound that.
    @pnomolos
    pnomolos committed Sep 16, 2022
    Configuration menu
    Copy the full SHA
    accaa5b View commit details
    Browse the repository at this point in the history

  2. Fix for Random.uuid not available in older Ruby versions.

    @pnomolos
    pnomolos committed Sep 16, 2022
    Configuration menu
    Copy the full SHA
    ea28153 View commit details
    Browse the repository at this point in the history