Commit changes to dist/index.js from latest dependency upgrades #184
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
It seems that several of the recent dependency upgrades courtesy of dependabot (e.g. 1402601 and adc83cb) have caused the committed `dist/index.js` file to become out of sync with the output of a fresh compilation. This commit updates `dist/index.js` with the output from running a fresh `npm install` and `npm run all`.
|
@rmacklin thanks for this PR! I apologize I am going to close it and link to the same thing but for one I generated locally. Just doing that so I can review/check it myself rather than try to review the diff and take a security risk on the distribution. Really appreciate it though! |
|
Ok, thanks! For posterity, the changes were identical (I'm glad to know our machines produced the same diff): |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
It seems that several of the recent dependency upgrades courtesy of dependabot (e.g. #155 and #163) have caused the committed
dist/index.jsfile to become out of sync with the output of a fresh compilation. This PR updatesdist/index.jswith the output from running a freshnpm installandnpm run all.This should make it so that future changes don't include large, unrelated diffs in
dist/index.js.