Skip to content
This repository has been archived by the owner on Feb 12, 2024. It is now read-only.

fix: fixing circuit-relaying example #1443

Merged
merged 1 commit into from
Jul 17, 2018
Merged

fix: fixing circuit-relaying example #1443

merged 1 commit into from
Jul 17, 2018

Conversation

dryajov
Copy link
Member

@dryajov dryajov commented Jul 16, 2018

No description provided.

@0x-r4bbit
Copy link
Contributor

@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?

@dryajov dryajov force-pushed the fix/circuit-example branch from 3b343ec to bc9d41d Compare July 16, 2018 20:10
@ghost ghost assigned dryajov Jul 16, 2018
@ghost ghost added the status/in-progress In progress label Jul 16, 2018
@dryajov
Copy link
Member Author

dryajov commented Jul 16, 2018

thanks for the heads up @PascalPrecht

},
"devDependencies": {
"aegir": "^14.0.0",
"http-server": "~0.10.0",
Copy link
Contributor

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)

Copy link
Member Author

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.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 got it!

@0x-r4bbit
Copy link
Contributor

@dryajov I left one more minor comment, sorry didn't catch this earlier.

@0x-r4bbit
Copy link
Contributor

This LGTM then

Copy link
Member

@alanshaw alanshaw left a 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!

@alanshaw alanshaw merged commit a681fc5 into master Jul 17, 2018
@ghost ghost removed the status/in-progress In progress label Jul 17, 2018
@alanshaw alanshaw deleted the fix/circuit-example branch July 17, 2018 12:24
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants