Skip to content

Fix export for TypeScript#66

Merged
JaneJeon merged 1 commit into
hanover-computing:masterfrom
Markus-Rost:patch-2
May 11, 2023
Merged

Fix export for TypeScript#66
JaneJeon merged 1 commit into
hanover-computing:masterfrom
Markus-Rost:patch-2

Conversation

@Markus-Rost

Copy link
Copy Markdown
Contributor

Needs to export the default to work properly, my bad.
Didn't notice earlier because I forgot to test it before the release.

Signed-off-by: MarkusRost <37275477+Markus-Rost@users.noreply.github.com>
@JaneJeon

Copy link
Copy Markdown
Collaborator

Oh, were you the guy who tried to make a PR previous about typescript something?

@JaneJeon

Copy link
Copy Markdown
Collaborator

Is there a way we can... idk, lint or "test" that the types are working properly within the repository? Maybe setup something w/ TS?

@Markus-Rost

Copy link
Copy Markdown
Contributor Author

I made the PR adding index.d.ts originally. I'm sadly not very familiar with TypeScript myself, I'm mostly trying to make VSC happy 😅

@Markus-Rost

Copy link
Copy Markdown
Contributor Author

But this time I at least tested the change properly :)

Previous export was just the Got type, now it's an object of the type and matching what you are importing from got in index.js

@JaneJeon
JaneJeon merged commit f62456c into hanover-computing:master May 11, 2023
@Markus-Rost
Markus-Rost deleted the patch-2 branch May 11, 2023 11:32
@JaneJeon

Copy link
Copy Markdown
Collaborator

Hello, could you verify that this release works for you when you import from it? https://github.com/hanover-computing/got-ssrf/releases/tag/v1.3.5

@Markus-Rost

Copy link
Copy Markdown
Contributor Author

Working for me (there were no code changes) and IDE is also happy about the typing of gotSsrf. I'm getting the same suggestions as just using plain got now so that should be fine as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants