-
Notifications
You must be signed in to change notification settings - Fork 99
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
deps(std): bump dependencies and prefer their lighter submodules #422
Conversation
Hey @BastiDood, thanks for the contribution, We'll try to review this and see if we can close it. |
Hey @BastiDood , any chance you might have to update to use the latest std (0.214.0)? |
Sounds great! Will definitely work on that over the weekend. I'll go rebase the branch while I'm at it. 👍 |
Hello there @bombillazo! It turns out I had some time to spare. This PR should be ready for testing now. |
Awesome, Ill take a look at it in the weekend |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good.
@bartlomieju wanna take a look? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I asked @iuioiua to take a look since he's been doing a lot of work on deno_std
, because of their LGTM, it also LGTM to me.
Thank you @BastiDood @bombillazo
This PR bumps the
std
dependencies from0.160.0
to the latest0.180.0
(as of writing). Breaking changes upstream (i.e., renamed exports, moved files, etc.) have been addressed accordingly.While I was bumping the versions, I also refactored some exports such that we now prefer importing submodules instead of the more general
mod.ts
files (which downloads a bunch of unneeded files).