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.

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

Use Clover's public python template #6

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

Conversation

wesleykendall
Copy link
Contributor

@dargueta CC @gwax this updates pytest-pgsql to use https://github.com/CloverHealth/temple-python-public as the template

The template was created using the scaffolding changes in this project, so there weren't a bunch of diffs. I manually updated some of the differences in the template that are not in this repo.

Doing this change means that pytest-pgsql will be kept in sync with changes to the template

@dargueta dargueta self-assigned this Aug 21, 2018
@jiffyclub
Copy link

You'll need to add installation of Postgres to the test setup (it's not part of the template), you can see an example here: https://github.com/CloverHealth/pgmock/blob/5c0d2be637c23729a306c0346b9ec1340450a54b/.circleci/config.yml#L17-L22.

@dargueta dargueta removed their assignment Jul 5, 2019
@dargueta
Copy link
Contributor

dargueta commented Jul 5, 2019

@jiffyclub I no longer have access to this branch as I was removed as a contributor. Unless Wes picks this back up again I think we're going to have to restart it.

Kinduva heretical thought -- do we need Temple?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants