-
-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
Postgres adapter #2012
Postgres adapter #2012
Conversation
@drew-gross updated the pull request. |
6 similar comments
@drew-gross updated the pull request. |
@drew-gross updated the pull request. |
@drew-gross updated the pull request. |
@drew-gross updated the pull request. |
@drew-gross updated the pull request. |
@drew-gross updated the pull request. |
Current coverage is 91.19%
@@ master #2012 diff @@
==========================================
Files 91 92 +1
Lines 6445 6517 +72
Methods 1108 1146 +38
Messages 0 0
Branches 1363 1368 +5
==========================================
+ Hits 5938 5943 +5
- Misses 507 574 +67
Partials 0 0
|
@drew-gross updated the pull request. |
4 similar comments
@drew-gross updated the pull request. |
@drew-gross updated the pull request. |
@drew-gross updated the pull request. |
@drew-gross updated the pull request. |
40723de
to
c308768
Compare
@drew-gross updated the pull request. |
@drew-gross updated the pull request. |
1 similar comment
@drew-gross updated the pull request. |
… stringified objects (parse-community#1922)"
7f8ff98
to
ba6f828
Compare
@drew-gross updated the pull request. |
@flovilmart said it was OK to merge :D |
|
||
addFieldIfNotExists(className, fieldName, type) { | ||
// TODO: Doing this in a transaction is probably a good idea. | ||
return this._client.query('ALTER TABLE "GameScore" ADD COLUMN "score" double precision', { className, fieldName }) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Uhm...
Ready to be merged after #2021 is merged