scalar / openapi-parser

Modern OpenAPI parser written in TypeScript
MIT License
26 stars 7 forks source link

fix: type import broken, fixes #133 #134

Closed hanspagel closed 1 month ago

hanspagel commented 1 month ago

I don’t know why the type import seems to work in the demo and in scalar/scalar, but the path is just wrong.

I just tried to find a way to catch this in CI, which seemed straight-forward, but it’s all just passing.

Anyway, here is the quick fix. I’ll find a solution to prevent this in the future.

changeset-bot[bot] commented 1 month ago

🦋 Changeset detected

Latest commit: 779d40865998211980fcf3865bf065e1cbf102a9

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package | Name | Type | | ---------------------- | ----- | | @scalar/openapi-parser | Patch |

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

relativeci[bot] commented 1 month ago

#102 Bundle Size — 114.45KiB (0%).

779d408(current) vs 21b6fc0 main#98(baseline)

Bundle metrics  no changes
                 Current
#102
     Baseline
#98
No change  Initial JS 114.45KiB 114.45KiB
No change  Initial CSS 0B 0B
No change  Cache Invalidation 0% 0%
No change  Chunks 45 45
No change  Assets 45 45
No change  Modules 45 45
No change  Duplicate Modules 0 0
No change  Duplicate Code 0% 0%
No change  Packages 0 0
No change  Duplicate Packages 0 0

Bundle size by type  no changes
|            |       Current
[#102](https://app.relative-ci.com/projects/hXQ54UQKluDUZAKvpt9b/jobs/102-tsVndetuDnekv1wGO2Sm?utm_source=github&utm_medium=pr-report "View bundle analysis report") |      Baseline
[#98](https://app.relative-ci.com/projects/hXQ54UQKluDUZAKvpt9b/jobs/98-AK38bFH6L3Gd2tbS4tVd?utm_source=github&utm_medium=pr-report "View baseline bundle analysis report") | |:--|--:|--:| | No change  [JS](https://app.relative-ci.com/projects/hXQ54UQKluDUZAKvpt9b/jobs/102-tsVndetuDnekv1wGO2Sm/assets?ba=%7B%22filters%22%3A%22ft.CSS-0_ft.JS-1_ft.IMG-0_ft.MEDIA-0_ft.FONT-0_ft.HTML-0_ft.OTHER-0%22%7D "View JS assets") | `114.45KiB` | `114.45KiB` |

Bundle analysis reportBranch fix/type-import-brokenProject dashboard

hanspagel commented 1 month ago

There are some weird issues with the dts plugin it seems. Some files land in random places in the dist folder.