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

Regression test for #871 #1283

Merged
merged 1 commit into from
Mar 30, 2016
Merged

Conversation

drew-gross
Copy link
Contributor

No description provided.

@flovilmart
Copy link
Contributor

Syntax error in your test

@facebook-github-bot
Copy link

@drew-gross updated the pull request.

let PersonObject = Parse.Object.extend("Person");
let p1 = new PersonObject();
let p2 = new PersonObject();
Parse.Object.saveAll([p1, p2]).then(([p1, p2]) => {
Copy link
Contributor

Choose a reason for hiding this comment

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

Causing a syntax error here

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Probably passed locally because I'm on a newer node. This is why we have CI :p

Copy link
Contributor

Choose a reason for hiding this comment

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

Yeah.... I get that often too. NVM FTW

@facebook-github-bot
Copy link

@drew-gross updated the pull request.

@codecov-io
Copy link

Current coverage is 93.00%

Merging #1283 into master will not affect coverage as of fd5c81d

@@            master   #1283   diff @@
======================================
  Files           84      84       
  Stmts         5275    5275       
  Branches       960     960       
  Methods          0       0       
======================================
  Hit           4906    4906       
  Partial         11      11       
  Missed         358     358       

Review entire Coverage Diff as of fd5c81d

Powered by Codecov. Updated on successful CI builds.

@flovilmart flovilmart merged commit 7afc08a into parse-community:master Mar 30, 2016
@drew-gross drew-gross deleted the test-871 branch March 30, 2016 23:31
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.

4 participants