Skip to content
This repository

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

constraint issues #4

Merged
merged 0 commits into from over 1 year ago

2 participants

Bradley Teller Rob Conery
Bradley Teller

This fix here is just an update to the sql script for postgresql to pull in the table names with primary keys. This doesn't address the issue for composite primary keys though. It seems either the recommendation would be to not use them, which seems less than ideal, or to change the way these arguments are composed and parsed.

Rob Conery
Owner

This sounds like a great idea - I can't read the diff though :/

Bradley Teller bteller merged commit 7ea49ef into from December 01, 2012
Bradley Teller bteller closed this December 01, 2012
Bradley Teller

I'm apparently making a mess of this thing. Initially it only shows the single line change, but after I commit it removes all 139 lines just to add a single line. This line below needs to be added under line 24. I'll keep trying.

and tc.constraint_type = 'PRIMARY KEY' \

Bradley Teller

Okay, I had to create a new pull request for this, but this time the diff is much cleaner.

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

Sorry, commit information is not available for this pull request.

This page is out of date. Refresh to see the latest.

Showing 0 changed files with 0 additions and 0 deletions. Show diff stats Hide diff stats

Commit_comment_tip

Tip: You can add notes to lines in a file. Hover to the left of a line to make a note

Something went wrong with that request. Please try again.