Merged
#29 · Created  · Last updated

Merged pull request

Merged in AE-5489 (pull request #29)

350e6d2·Author: ·Closed by: ·2017-12-14

Description

  • Update test to handle latest postgres version

    The error message changed slightly

    AE-5489 #time 10m

  • Pull the test DB URI from the environment

    So we don't have to hard-code my credentials in my local dev anymore

    AE-5489 #time 5m

  • Add a simple template for testing

    I'm using this now so I don't have to run a Postgres instance locally

    AE-5489 #time 5m

  • Allow using multiple ordering clauses

    We need this so that we can pass in the sepiida's request.sorts parameter as our orders parameter and have a 1-to-1 mapping between the two. This allows chryso to be standalone because it has its own implementation of the OrderParameter as well as to work seamlessly with sepiida in cases where the client wants both of them by making the two classes identical in structure.

    This also avoids forcing the orders list to be a list of strings. They can be, but if they aren't and they conform to our class structure we just use them as-is and avoid additional parsing

    AE-5489 #time 30m

  • Add my pylint.fish script for running pylint easily

    It's short, but gets the job done

    AE-5489 #time 2m

  • Fix various pylint errors

    Thanks latest version of pylint!

    AE-5489 #time 3m

  • Break out changelog, add 2.0 notes

    AE-5489 #time 5m

0 attachments

0 comments

Loading commits...