update npm package dependencies #9
Merged
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
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.
The main prompt for this PR is to fix the security vulnerability caused by the outdated
ecstaticpackage, but since I was already in there, I updated all of the other npm packages that were listed innpm outdated.I also did these things:
Replace the
mythpackage withmyth6because of this issue: obj.hasOwnProperty is not a function segmentio/myth#149.Fix the name of this app in the package.json file. It must have initially been called
slidedeck.Update the license to an official SPDX code.
Add a
console.logto the server startup to keep from having to look intoserver.jsto know what port the service is running onRemove the test section from the README because there are no tests to run.
I'm attaching 3 reviewers, but 1 should suffice. I don't have access.
Don't forget to deploy after you merge this.