Commit fcb5b1b
authored
fix: replace inline type modifiers with export type block (#1062)
## Summary
- `rollup-plugin-typescript2` with rollup v3 fails to parse the inline
`type` modifier in named export lists (e.g. `export { type Foo, Bar }`)
— a TypeScript 4.5 feature not understood by rollup v3's acorn parser
- This caused the CI build to fail with `RollupError: Unexpected token`
at `src/index.tsx:45`
- Moves all type-only exports from inline modifiers into a separate
`export type { }` statement (TypeScript 3.8, fully supported by the
current toolchain)
- Public API is unchanged — all the same types are exported
## Test plan
- [ ] `npm run build` passes locally
- [ ] CI passes on this PR
- [ ] After merging, comment `@dependabot rebase` on #1061 so it picks
up this fix1 parent 5ee2fef commit fcb5b1b
1 file changed
+19
-17
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
42 | 42 | | |
43 | 43 | | |
44 | 44 | | |
45 | | - | |
46 | 45 | | |
47 | 46 | | |
48 | 47 | | |
49 | 48 | | |
50 | | - | |
51 | 49 | | |
52 | 50 | | |
53 | 51 | | |
54 | | - | |
55 | | - | |
56 | | - | |
57 | | - | |
58 | | - | |
59 | | - | |
60 | | - | |
61 | | - | |
62 | | - | |
63 | | - | |
64 | | - | |
65 | | - | |
66 | | - | |
67 | | - | |
68 | | - | |
69 | 52 | | |
70 | 53 | | |
71 | 54 | | |
| |||
74 | 57 | | |
75 | 58 | | |
76 | 59 | | |
| 60 | + | |
| 61 | + | |
| 62 | + | |
| 63 | + | |
| 64 | + | |
| 65 | + | |
| 66 | + | |
| 67 | + | |
| 68 | + | |
| 69 | + | |
| 70 | + | |
| 71 | + | |
| 72 | + | |
| 73 | + | |
| 74 | + | |
| 75 | + | |
| 76 | + | |
| 77 | + | |
| 78 | + | |
77 | 79 | | |
0 commit comments