Skip to content

Commit

Permalink
fix(cli): try to resolve `Cannot read properties of undefined (readin…
Browse files Browse the repository at this point in the history
…g 'includes')` error (#4675)
  • Loading branch information
dsinghvi authored Sep 17, 2024
1 parent a97860e commit 40c799a
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 1 deletion.
2 changes: 1 addition & 1 deletion packages/cli/cli/src/rerunFernCliAtVersion.ts
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ export async function rerunFernCliAtVersion({
[FERN_CWD_ENV_VAR]: process.env[FERN_CWD_ENV_VAR] ?? process.cwd()
}
});
if (stdout.includes("code EEXIST") || stderr.includes("code EEXIST")) {
if (stdout?.includes("code EEXIST") || stderr?.includes("code EEXIST")) {
// try again if there is a npx conflict
return await rerunFernCliAtVersion({
version,
Expand Down
8 changes: 8 additions & 0 deletions packages/cli/cli/versions.yml
Original file line number Diff line number Diff line change
@@ -1,3 +1,11 @@
- changelogEntry:
- summary: |
Make sure to check for optionality when parsing stdout and stderr in CLI. This
removes the error: `Cannot read properties of undefined (reading 'includes')`.
type: fix
irVersion: 53
version: 0.42.1

- changelogEntry:
- summary: |
If you merge multiple OpenAPI specs with namespaces, `fern check` will no longer
Expand Down

0 comments on commit 40c799a

Please sign in to comment.