Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: export all types from client with S3 prefix #38

Merged
merged 3 commits into from
May 10, 2024

Conversation

karfau
Copy link
Contributor

@karfau karfau commented May 4, 2024

to make it possible to refer to the type of the constructor arguments from the outside without any "hacks".

@karfau
Copy link
Contributor Author

karfau commented May 6, 2024

I'm assuming the failing check is unrelated to my changes?

@bradenmacdonald
Copy link
Owner

Yeah, looks unrelated. I'll take a look when I can.
Thanks for this contribution, btw. I'll merge it as soon as I find time to resolve the failing check, merge your PR, and publish a new version.

@karfau
Copy link
Contributor Author

karfau commented May 6, 2024

In the meantime I was also thinking about exporting more types, especially the return types like S3Object.
Would you be fine if I add them as part of this PR?

@bradenmacdonald
Copy link
Owner

Sure, go for it!

@karfau karfau changed the title feat: export S3ClientOptions from mod.ts feat: export all types from client with S3 prefix May 7, 2024
@bradenmacdonald bradenmacdonald merged commit ff05c48 into bradenmacdonald:main May 10, 2024
2 checks passed
@karfau karfau deleted the export-options branch May 10, 2024 06:28
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