-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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
chore: update jsii to 0.13.2 #3141
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.
I fixed the .npmignore files (just removed the .jsii line instead of a new "!.jsii" line).
Would be good to enforce this with .npmignore...
Also, @rix0rrr if the new version already includes all the fixes to vendored_deps/jsii-diff-0.13.x.tgz
, then we can just remove it (or does it still contain some changes that are not published, in which case we shouldn't override it here).
|
||
# Include .jsii | ||
!.jsii |
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 odd... we have .jsii excluded and then included? It should be included (!!)
Can we enforce with pkglint?
Since "master" is broken right now, I'll merge this in and we should follow up with a pkglint fix. |
Please read the contribution guidelines and follow the pull-request checklist.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license