Promise catch syntax error

[eddiezane]

This is on the develop branch.

returnsAPromise()
  .catch(err => {
    console.error(err)
  })

[amadeus]

Awesome, thanks the find. I'll look into fixing this shortly!

[eddiezane]

@amadeus, @bounceme, and @davidchambers - y'all are the best people in the world. Such a stunning display of open source project management 💖 .

[alexlafroscia]

I was just coming over to this repo to report the same thing! A few more examples in my screenshot:

It seems like a "regular" function passed to .catch gets thrown off as well.

[amadeus]

Yeah it's because catch is getting matched as a catch statement, not as a function call.

[alexlafroscia]

Totally makes sense, it's tough (and unfortunate) that it's both a keyword and the way to handle Promise errors.

As always, thanks for the hard work and attentive response to issues @amadeus!

[amadeus]

I have a fix in #485, perhaps check it out and confirm that it fixes the issue. Also make sure I didn't drastically break something else either xD

[amadeus]

Should be fixed in develop

[alexlafroscia]

@amadeus looks good to me! .catch is now un-highlighted when used as a method