feat(reportError): fix linting & release reportError

Author: BrunoScheufler

package.json

@@ -30,27 +30,27 @@
     }
   ],
   "devDependencies": {
-    "@sentry/node": "4.5.3",
-    "@types/lodash": "4.14.120",
-    "@types/node": "10.12.18",
-    "graphql": "14.1.1",
-    "graphql-middleware": "3.0.0",
-    "prettier": "1.16.1",
+    "@sentry/node": "5.2.0",
+    "@types/lodash": "4.14.130",
+    "@types/node": "12.0.2",
+    "graphql": "14.3.0",
+    "graphql-middleware": "3.0.2",
+    "prettier": "1.17.1",
     "prettier-check": "2.0.0",
     "rimraf": "2.6.3",
-    "semantic-release": "15.13.3",
-    "tslint": "5.12.1",
-    "tslint-config-prettier": "1.17.0",
+    "semantic-release": "15.13.12",
+    "tslint": "5.16.0",
+    "tslint-config-prettier": "1.18.0",
     "tslint-config-standard": "8.0.1",
-    "typescript": "3.2.4"
+    "typescript": "3.4.5"
   },
   "license": "MIT",
   "repository": {
     "type": "git",
     "url": "https://github.com/BrunoScheufler/graphql-middleware-sentry.git"
   },
   "peerDependencies": {
-    "@sentry/node": "^4.5.0",
+    "@sentry/node": "^5.2.0",
     "graphql": "^0.11.0 || ^0.12.0 || ^0.13.0 || ^14.0.0",
     "graphql-middleware": "^2.0.0 || ^3.0.0"
   }

src/index.ts

@@ -6,7 +6,7 @@ export type ExceptionScope<Context> = (
   scope: Sentry.Scope,
   error: Error,
   context: Context,
-  reportError?: (res: Error | any) => boolean
+  reportError?: (res: Error | any) => boolean,
 ) => void
 
 // Options for graphql-middleware-sentry
@@ -27,7 +27,7 @@ export const sentry = <Context>({
   withScope,
   captureReturnedErrors = false,
   forwardErrors = false,
-  reportError
+  reportError,
 }: Options<Context>): IMiddlewareFunction => {
   // Check if either sentryInstance or config.dsn is present
   if (!sentryInstance && !config.dsn) {
@@ -70,7 +70,7 @@ function captureException<Context>(
   withScope: ExceptionScope<Context>,
   reportError?: (res) => boolean,
 ) {
-  if (reportError && reportError(error) || reportError === undefined) {
+  if ((reportError && reportError(error)) || reportError === undefined) {
     sentryInstance.withScope(scope => {
       withScope(scope, error, ctx)
       sentryInstance.captureException(error)