-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
[Data Plane] pins rollup to 1.13.1 #3494
Conversation
724032b
to
083496f
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.
LGTM, might want to wait for Eng Sys to review @mikeharder @bsiegel
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.
Since rollup is a direct dependency, we should change our package.json files to simply reflect the version requirement we want:
"rollup": "~1.13.1"
Rather than using the global override mechanism provided by preferredVersions
. That should only be used when we need to pin a transitive dependency we have no direct control over.
@bsiegel |
3f7b07b
to
123ea67
Compare
@bsiegel |
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.
👍✨Ship It✨
Rollup versions >= 1.14.0 introduced a change that is causing our test rollup bundle to break. This version was released earlier today and we're specifically seeing integration test failures in
@azure/event-bus
.This change pins rollup to version
1.13.1
, which gets integration tests back to a mostly working state.See #3484 for more information.