From d999a05a5c3a9b649907864a86ad1e877d3f2e8e Mon Sep 17 00:00:00 2001 From: Alex Freska Date: Mon, 21 Oct 2024 15:59:32 -0400 Subject: [PATCH] chore: clean up logs remove unnecessary override --- apps/website/content/prs.ts | 6 +++++- libs/request/src/index.ts | 3 +-- 2 files changed, 6 insertions(+), 3 deletions(-) diff --git a/apps/website/content/prs.ts b/apps/website/content/prs.ts index 026c706f4..a011d3ed9 100644 --- a/apps/website/content/prs.ts +++ b/apps/website/content/prs.ts @@ -23,7 +23,11 @@ export async function getPrs() { }, }) } catch (e) { - console.log(e) + // Occasionally there is an edge-case error such as: + // "Could not parse expression with acorn: Unexpected token" + // Simply procceed with source=null in these cases. + // We can revisit this next when we do the next website refactor. + // Updating all the mdx packages might even do the trick. } return { source, diff --git a/libs/request/src/index.ts b/libs/request/src/index.ts index 1875ddb8c..81afd4a12 100644 --- a/libs/request/src/index.ts +++ b/libs/request/src/index.ts @@ -84,8 +84,7 @@ export function buildRequestHandler< ...mergedArgs.params, } - // eslint-disable-next-line @typescript-eslint/no-non-null-assertion - const paramRoute = parameterizeRoute(route, params as RequestParams)! + const paramRoute = parameterizeRoute(route, params as RequestParams) const data = 'data' in mergedArgs ? mergedArgs.data : undefined