-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Conversation
@dryajov If you put "Fixes #1423" into the commit message body a la https://github.com/ipfs/community/blob/master/js-code-guidelines.md#examples, the corresponding issue gets closed automatically, once this PR lands :) Maybe you can update this commit? |
3b343ec
to
bc9d41d
Compare
thanks for the heads up @PascalPrecht |
}, | ||
"devDependencies": { | ||
"aegir": "^14.0.0", | ||
"http-server": "~0.10.0", |
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.
One thing though: Why exactly is this going to be removed? Maybe it shouldn't have been there in the first place, but I assume this has been put there for a reason and it doesn't seem to be necessary for this commit to do its work (fixing that example)
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.
It was missed. I left it there because the example was initially based on browserify and later moved to parcel.
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.
👍 got it!
@dryajov I left one more minor comment, sorry didn't catch this earlier. |
This LGTM then |
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.
It works for me!
No description provided.