-
Notifications
You must be signed in to change notification settings - Fork 64
fix: clean up blob directories in pre-dev #5600
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
Conversation
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, but I think there's a promise issue.
8fa37d0
to
4ade4b0
Compare
This pull request adds or modifies JavaScript ( |
28bc631
to
8dc9982
Compare
3276df8
to
e759131
Compare
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 great! Left one tiny comment.
55d3f38
to
04eb7b1
Compare
04eb7b1
to
059419e
Compare
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 great! Left one tiny nit, feel free to take it or leave it.
Summary
Blobs directories are interfering with
ntl dev
innext-runtime-minimal
, specifically around theexperimentalRegion
flag. These changes clear the blob directories before runningntl dev
. We already do this pre-build, we would now also be doing it pre-dev.For us to review and ship your PR efficiently, please perform the following steps:
we can discuss the changes and get feedback from everyone that should be involved. If you`re fixing a typo or
something that`s on fire 🔥 (e.g. incident related), you can skip this step.
your code follows our style guide and passes our tests.