Skip to content
This repository was archived by the owner on Aug 28, 2024. It is now read-only.

chore: loosen the types #117

Closed
wants to merge 2 commits into from
Closed

chore: loosen the types #117

wants to merge 2 commits into from

Conversation

hanspagel
Copy link
Member

We want to type the specification we get from the user so we have an autocomplete in the IDE. But we also have to expect that the user doesn’t adhere to the OpenAPI specification, so any attribute can be passed on any level.

This isn’t as beautiful as it could be, but works for now. 😬 I’ll go over the types again ASAP.

Copy link

changeset-bot bot commented Jun 5, 2024

🦋 Changeset detected

Latest commit: cc3b850

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

@hanspagel hanspagel requested a review from marclave June 5, 2024 14:37
Copy link

relativeci bot commented Jun 5, 2024

#71 Bundle Size — 114.09KiB (0%).

cc3b850(current) vs 05dbbc4 main#68(baseline)

Bundle metrics  no changes
                 Current
#71
     Baseline
#68
No change  Initial JS 114.09KiB 114.09KiB
No change  Initial CSS 0B 0B
No change  Cache Invalidation 0% 0%
No change  Chunks 44 44
No change  Assets 44 44
No change  Modules 44 44
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
#71
     Baseline
#68
No change  JS 114.09KiB 114.09KiB

Bundle analysis reportBranch fix/typesProject dashboard

@hanspagel hanspagel marked this pull request as draft June 5, 2024 18:30
@hanspagel
Copy link
Member Author

I don’t like this solution. Need to think about what we really want.

@hanspagel hanspagel closed this Jun 6, 2024
@hanspagel hanspagel deleted the fix/types branch June 21, 2024 12:20
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant