tgriesser / knex / 2825

88%

push

travis-ci

<a href="https://github.com/tgriesser/knex/commit/<a class=hub.com/tgriesser/knex/commit/ee8cc35ecb41a8bc809f753b56e64691bd3fa25c">ee8cc35ec<a href="https://github.com/tgriesser/knex/commit/ee8cc35ecb41a8bc809f753b56e64691bd3fa25c">&quot;&gt;Fix #2545, bring redshift wrapIdentifier custom hook up to date with postgres (#2551)

* Add a Redshift dialect that inherits from Postgres.

* Turn .index() and .dropIndex() into no-ops with warnings in the Redshift dialect.

* Update the Redshift dialect to be compatible with master.

* Update package.json

* Disable liftoff cli

* Remove the CLI

* Add lib to the repo

* Allow the escaping of named bindings.

* Update dist

* Update the Redshift dialect’s instantiation of the query and column compilers.

* Update the distribution

* Fix a merge conflict

* Take lib back out

* Trying to bring back in line with tgreisser/knex

* Add npm 5 package-lock

* Bring cli.js back in line

* Bring cli.js back in line

* Progress commit on redshift integration tests

* Revert &amp;quot;Progress commit on redshift integration tests&quot;

This reverts commit </a><a class="double-link" href="https://github.com/tgriesser/knex/commit/<a class="double-link" href="https://github.com/tgriesser/knex/commit/207e31635c638853dec54ce0580d34559ba5a54c">207e31635</a>">207e31635</a><a href="https://github.com/tgriesser/knex/commit/ee8cc35ecb41a8bc809f753b56e64691bd3fa25c">.

* Progress commit

* Working not null on primary columns in createTable

* Working redshift unit tests

* Working unit and integration tests, still need to fix migration tests

* Brought datatypes more in line with what redshift actually supports

* Added query compiler unit tests

* Add a hacky returning clause for redshift ugh

* Working migration integration tests

* Working insert integration tests

* Allow multiple insert returning values

* Working select integration tests

* Working join integration tests

* Working aggregate integration tests

* All integration suite tests working

* Put docker index for reconnect tests back

* Redshift does not support insert...returning, there does not seem to be a way around that, theref... (continued)

2215 of 2996 branches covered (73.93%)

5250 of 5899 relevant lines covered (89.0%)

2953.85 hits per line