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

Fixed command examples #244

Merged
merged 2 commits into from
Nov 21, 2016
Merged

Fixed command examples #244

merged 2 commits into from
Nov 21, 2016

Conversation

dustinmoorenet
Copy link
Contributor

--connection parameter needs to be present for connection string otherwise default connection is made

`--connection` parameter needs to be present for connection string otherwise default connection is made
Copy link
Member

@benjie benjie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 Thanks for spotting and fixing this

@benjie
Copy link
Member

benjie commented Nov 21, 2016

(Just waiting for the build to pass so I can merge it)

@benjie
Copy link
Member

benjie commented Nov 21, 2016

It seems Travis had a bit of a backlog issue earlier, there's no indication it received the webhook for this push.

https://www.traviscistatus.com/

I'm going to edit a character to trigger a rebuild.

@benjie
Copy link
Member

benjie commented Nov 21, 2016

@benjie
Copy link
Member

benjie commented Nov 21, 2016

Ohhhhhhhhhhhhhh

My bad. I broke the config.

@benjie benjie merged commit 4c686a7 into graphile:master Nov 21, 2016
@benjie
Copy link
Member

benjie commented Nov 21, 2016

In conclusion, for tests to run against master for greenkeeper we need "build pushes" turned on; for tests to run on remote pull requests we need "build pull requests" turned on. For local pull requests (e.g. from contributors) every commit in a PR will thus be built twice and there doesn't seem to be a way to prevent that.

Belline pushed a commit to Belline/postgraphql that referenced this pull request Dec 18, 2017
`--connection` parameter needs to be present for connection string otherwise default connection is made
benjie pushed a commit that referenced this pull request Jan 27, 2020
Fixes #244 

Uses a template string to support multiple PostgreSQL versions.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants