Skip to content

Commit

Permalink
Fix astro add adding lines from extended configurations (#5095)
Browse files Browse the repository at this point in the history
  • Loading branch information
Princesseuh authored Oct 14, 2022
1 parent 0173c2b commit ddfbef5
Show file tree
Hide file tree
Showing 4 changed files with 14 additions and 1 deletion.
5 changes: 5 additions & 0 deletions .changeset/gorgeous-baboons-sleep.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
'astro': patch
---

Fix `astro add` trying to add lines from extended configurations when adding frameworks
5 changes: 5 additions & 0 deletions .changeset/neat-phones-sin.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
'@astrojs/react': patch
---

Add `@types/` packages as peerDependencies
2 changes: 2 additions & 0 deletions packages/astro/src/core/config/tsconfig.ts
Original file line number Diff line number Diff line change
Expand Up @@ -56,6 +56,7 @@ export function loadTSConfig(cwd: string | undefined, resolve = true): tsr.TsCon
let config = tsr.tsconfigResolverSync({
cwd,
filePath: resolve ? undefined : cwd,
ignoreExtends: !resolve,
});

// When a direct filepath is provided to `tsconfigResolver`, it'll instead return invalid-config even when
Expand All @@ -70,6 +71,7 @@ export function loadTSConfig(cwd: string | undefined, resolve = true): tsr.TsCon
cwd,
filePath: resolve ? undefined : cwd,
searchName: 'jsconfig.json',
ignoreExtends: !resolve,
});

if (
Expand Down
3 changes: 2 additions & 1 deletion packages/integrations/react/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,8 @@
"peerDependencies": {
"react": "^17.0.2 || ^18.0.0",
"react-dom": "^17.0.2 || ^18.0.0",
"@types/react": "^17.0.50 || ^18.0.21"
"@types/react": "^17.0.50 || ^18.0.21",
"@types/react-dom": "^17.0.17 || ^18.0.6"
},
"engines": {
"node": "^14.18.0 || >=16.12.0"
Expand Down

0 comments on commit ddfbef5

Please sign in to comment.