Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

fix for 'spec' is not defined #43

Merged
merged 1 commit into from Aug 28, 2012

Conversation

Projects
None yet
2 participants
Contributor

ryanmeador commented Aug 28, 2012

I was having a problem using node-db-migrate with a Postgres backend to change a column, adding a default value that was a number. The problem is setDefaultValue() is referencing a variable called 'spec', but there is no such variable in the scope. I'm pretty sure this was supposed to be 'columnSpec', and I changed it to that and it appears to work. Please pull this change into your repo.

@kunklejr kunklejr added a commit that referenced this pull request Aug 28, 2012

@kunklejr kunklejr Merge pull request #43 from lede/master
fix for 'spec' is not defined
7236767

@kunklejr kunklejr merged commit 7236767 into db-migrate:master Aug 28, 2012

Contributor

kunklejr commented Aug 28, 2012

Fix is now available in version 0.2.2 via npm. Thanks for the pull request!

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