piccolo icon indicating copy to clipboard operation
piccolo copied to clipboard

CockroachDB Support

Open gnat opened this issue 2 years ago • 38 comments

Work in progress for https://github.com/piccolo-orm/piccolo/issues/607

gnat avatar Aug 31 '22 21:08 gnat

This pull request introduces 9 alerts when merging 3f050fc518dbeba8c49959cddfa56bf9877768d4 into ed93feb85050895cc54e7db9650eddc69ab94fa0 - view on LGTM.com

new alerts:

  • 5 for Unused import
  • 2 for Wrong number of arguments in a call
  • 2 for Wrong number of arguments in a class instantiation

lgtm-com[bot] avatar Aug 31 '22 22:08 lgtm-com[bot]

This pull request introduces 9 alerts when merging 18e27c1867cd6440c6e742c021710fbbef65488a into ed93feb85050895cc54e7db9650eddc69ab94fa0 - view on LGTM.com

new alerts:

  • 5 for Unused import
  • 2 for Wrong number of arguments in a call
  • 2 for Wrong number of arguments in a class instantiation

lgtm-com[bot] avatar Sep 01 '22 00:09 lgtm-com[bot]

This pull request introduces 7 alerts when merging 177ed058223ed1dce97ea0ceecf7fcfc41d970bf into ed93feb85050895cc54e7db9650eddc69ab94fa0 - view on LGTM.com

new alerts:

  • 7 for Unused import

lgtm-com[bot] avatar Sep 01 '22 10:09 lgtm-com[bot]

This pull request introduces 7 alerts when merging 22b88ade381429da0fbf795efd9779dfc21bf6f7 into ed93feb85050895cc54e7db9650eddc69ab94fa0 - view on LGTM.com

new alerts:

  • 6 for Unused import
  • 1 for First argument to super() is not enclosing class

lgtm-com[bot] avatar Sep 01 '22 13:09 lgtm-com[bot]

This pull request introduces 7 alerts when merging c9af100773ce9c1e013e3095a099105c5dfe324c into ed93feb85050895cc54e7db9650eddc69ab94fa0 - view on LGTM.com

new alerts:

  • 6 for Unused import
  • 1 for First argument to super() is not enclosing class

lgtm-com[bot] avatar Sep 01 '22 15:09 lgtm-com[bot]

This pull request introduces 7 alerts when merging 62be177507349373bcfc31825eb51a44a752fe2f into ed93feb85050895cc54e7db9650eddc69ab94fa0 - view on LGTM.com

new alerts:

  • 6 for Unused import
  • 1 for First argument to super() is not enclosing class

lgtm-com[bot] avatar Sep 01 '22 15:09 lgtm-com[bot]

This pull request introduces 8 alerts when merging 14c0cf2b8f890efef66f26ee42d42c5f89b3e434 into 7754498705540c81c40ec203a7bb429d59a07da5 - view on LGTM.com

new alerts:

  • 6 for Unused import
  • 1 for Unused local variable
  • 1 for First argument to super() is not enclosing class

lgtm-com[bot] avatar Sep 12 '22 05:09 lgtm-com[bot]

This pull request introduces 2 alerts when merging a52596f2bcee0f41c405074e68a459a67fa3172b into 7754498705540c81c40ec203a7bb429d59a07da5 - view on LGTM.com

new alerts:

  • 1 for Unused local variable
  • 1 for First argument to super() is not enclosing class

lgtm-com[bot] avatar Sep 12 '22 05:09 lgtm-com[bot]

This pull request introduces 2 alerts when merging e1d56fca2a46b93e7f5d2aef7abcc4e218c2a5fb into 7754498705540c81c40ec203a7bb429d59a07da5 - view on LGTM.com

new alerts:

  • 1 for Unused local variable
  • 1 for First argument to super() is not enclosing class

lgtm-com[bot] avatar Sep 12 '22 09:09 lgtm-com[bot]

This pull request introduces 2 alerts when merging 82076151874aa2d64fbfe34ee5393580375e5cd7 into 7754498705540c81c40ec203a7bb429d59a07da5 - view on LGTM.com

new alerts:

  • 1 for Unused local variable
  • 1 for First argument to super() is not enclosing class

lgtm-com[bot] avatar Sep 12 '22 10:09 lgtm-com[bot]

Thank you for the refinements and guidance so far @dantownsend !

gnat avatar Sep 12 '22 12:09 gnat

This pull request introduces 2 alerts when merging 3a96449e1bd8cb29484dbec24e7b6086a1b83670 into 7754498705540c81c40ec203a7bb429d59a07da5 - view on LGTM.com

new alerts:

  • 1 for Unused local variable
  • 1 for First argument to super() is not enclosing class

lgtm-com[bot] avatar Sep 12 '22 12:09 lgtm-com[bot]

This pull request introduces 2 alerts when merging 420ca4a173d7fd1cb4952e810eb97164088eb4b2 into 7754498705540c81c40ec203a7bb429d59a07da5 - view on LGTM.com

new alerts:

  • 1 for Unused local variable
  • 1 for First argument to super() is not enclosing class

lgtm-com[bot] avatar Sep 12 '22 13:09 lgtm-com[bot]

This pull request introduces 2 alerts when merging 93aec684ed8635e28d0627f841ffe8cc4c45b58f into 7754498705540c81c40ec203a7bb429d59a07da5 - view on LGTM.com

new alerts:

  • 1 for Unused local variable
  • 1 for First argument to super() is not enclosing class

lgtm-com[bot] avatar Sep 12 '22 13:09 lgtm-com[bot]

This pull request introduces 2 alerts when merging 183a9da67bdb796a9788fd872a44520ff6ad35d0 into 7754498705540c81c40ec203a7bb429d59a07da5 - view on LGTM.com

new alerts:

  • 1 for Unused local variable
  • 1 for First argument to super() is not enclosing class

lgtm-com[bot] avatar Sep 12 '22 14:09 lgtm-com[bot]

This pull request introduces 2 alerts when merging 49a238193ca40b30d70ece351e70bdc1b029bc69 into 7754498705540c81c40ec203a7bb429d59a07da5 - view on LGTM.com

new alerts:

  • 1 for Unused local variable
  • 1 for First argument to super() is not enclosing class

lgtm-com[bot] avatar Sep 16 '22 03:09 lgtm-com[bot]

This pull request introduces 2 alerts when merging c53b06b029d3cfac5590aab494d151537fdd80a6 into 7754498705540c81c40ec203a7bb429d59a07da5 - view on LGTM.com

new alerts:

  • 1 for Unused local variable
  • 1 for First argument to super() is not enclosing class

lgtm-com[bot] avatar Sep 19 '22 05:09 lgtm-com[bot]

This pull request introduces 2 alerts when merging 50aaa8bef19de0fc133e4d2221fa686a08df99af into 7754498705540c81c40ec203a7bb429d59a07da5 - view on LGTM.com

new alerts:

  • 1 for Unused local variable
  • 1 for First argument to super() is not enclosing class

lgtm-com[bot] avatar Sep 19 '22 17:09 lgtm-com[bot]

This pull request introduces 2 alerts when merging 367705e85e9cd992c26501582e8793708d8ed528 into 7754498705540c81c40ec203a7bb429d59a07da5 - view on LGTM.com

new alerts:

  • 1 for Unused local variable
  • 1 for First argument to super() is not enclosing class

lgtm-com[bot] avatar Sep 19 '22 18:09 lgtm-com[bot]

This pull request introduces 2 alerts when merging dcdf2228994208c6a1db1aae59366edd7f69f65c into 7754498705540c81c40ec203a7bb429d59a07da5 - view on LGTM.com

new alerts:

  • 1 for Unused local variable
  • 1 for First argument to super() is not enclosing class

lgtm-com[bot] avatar Sep 20 '22 11:09 lgtm-com[bot]

This pull request introduces 2 alerts when merging 8f0b50f8762f8daed160d2f571f7b5d475b92bd7 into 7754498705540c81c40ec203a7bb429d59a07da5 - view on LGTM.com

new alerts:

  • 1 for Unused local variable
  • 1 for First argument to super() is not enclosing class

lgtm-com[bot] avatar Sep 20 '22 14:09 lgtm-com[bot]

This pull request introduces 2 alerts when merging 809ffd749534639c20a3b0ad5446d9ab415e251c into 7754498705540c81c40ec203a7bb429d59a07da5 - view on LGTM.com

new alerts:

  • 1 for Unused local variable
  • 1 for First argument to super() is not enclosing class

lgtm-com[bot] avatar Sep 20 '22 22:09 lgtm-com[bot]

This pull request introduces 2 alerts when merging ccd1abbee68a3c5f6292cbc279dd1ab8a3c439a5 into 7754498705540c81c40ec203a7bb429d59a07da5 - view on LGTM.com

new alerts:

  • 1 for Unused local variable
  • 1 for First argument to super() is not enclosing class

lgtm-com[bot] avatar Sep 20 '22 23:09 lgtm-com[bot]

This pull request introduces 2 alerts when merging 5427a1cc345849cf19dd2e6f98a657bd56dc7743 into 7754498705540c81c40ec203a7bb429d59a07da5 - view on LGTM.com

new alerts:

  • 1 for Unused local variable
  • 1 for First argument to super() is not enclosing class

lgtm-com[bot] avatar Sep 20 '22 23:09 lgtm-com[bot]

This pull request introduces 1 alert when merging ea369886a442fd840901e1f4e5b7b97de646753b into 7754498705540c81c40ec203a7bb429d59a07da5 - view on LGTM.com

new alerts:

  • 1 for First argument to super() is not enclosing class

lgtm-com[bot] avatar Sep 21 '22 00:09 lgtm-com[bot]

This pull request introduces 1 alert when merging 2a44cc23c659457d8d741c4e06ee44f83b1373a3 into 7754498705540c81c40ec203a7bb429d59a07da5 - view on LGTM.com

new alerts:

  • 1 for First argument to super() is not enclosing class

lgtm-com[bot] avatar Sep 21 '22 00:09 lgtm-com[bot]

This pull request introduces 1 alert when merging faaddd46e7b596d0bd26f7cdacb76fd8fa1ea779 into 7754498705540c81c40ec203a7bb429d59a07da5 - view on LGTM.com

new alerts:

  • 1 for First argument to super() is not enclosing class

lgtm-com[bot] avatar Sep 21 '22 00:09 lgtm-com[bot]

This pull request introduces 2 alerts when merging dd1366697186fc23e54d0454ad008364fb891af8 into 7754498705540c81c40ec203a7bb429d59a07da5 - view on LGTM.com

new alerts:

  • 1 for Unreachable code
  • 1 for First argument to super() is not enclosing class

lgtm-com[bot] avatar Sep 21 '22 01:09 lgtm-com[bot]

This pull request introduces 1 alert when merging 47742bb6fe4f2c205efccf89584cac9637e0209b into 7754498705540c81c40ec203a7bb429d59a07da5 - view on LGTM.com

new alerts:

  • 1 for First argument to super() is not enclosing class

lgtm-com[bot] avatar Sep 21 '22 01:09 lgtm-com[bot]

This pull request introduces 1 alert when merging 5a8b6b039277fda957e6cd6bc69ac1ba7a84aa7a into 7754498705540c81c40ec203a7bb429d59a07da5 - view on LGTM.com

new alerts:

  • 1 for First argument to super() is not enclosing class

lgtm-com[bot] avatar Sep 21 '22 01:09 lgtm-com[bot]