Codesandbox-client: How to point dependency to a github branch

Created on 14 Jan 2019  ·  6Comments  ·  Source: codesandbox/codesandbox-client

❔ Question

Hi,
I would like to point my dependency to a branch on github, rather than a particular version. In my package.json, I've pointed it like this :

"react-timeseries-charts": "esnet/react-timeseries-charts#fix-355"

However, this doesn't work and I get this message

Could not fetch dependencies, please try again in a couple seconds: Unexpected token < in JSON at position 0

I saw an issue (#88) that suggests that this should work. Could you point me as to how to go about this?

Link to sandbox: link

Your Environment

| Software | Name/Version|
| ---------------- | ---------- |
| Сodesandbox |
| Browser | Chrome
| Operating System | macOS Sierra

Packager stale 🎒 Bundler

Most helpful comment

It's possible to install GitHub dependencies. Keep in mind that we just serve the source files of the GitHub repo (like npm and yarn), we do run the postinstall script, so if you want to build some files you can put it in that step.

Regarding esnet/react-timeseries-charts#fix-355, I just found out that our caching system doesn't support # in the paths, that's why it's showing those errors. I can take a look at that.

Here's the example I'm working on: codesandbox.io/s/z6j7954jnx

I get a different error in that sandbox, about missing regeneratorRuntime. ~I think that's because babel-polyfill is not required in the project. Going to double check that.~

Never mind, that's an issue on our side. Going to take a look!

Committed a fix on master for the regeneratorRuntime error.

All 6 comments

I'm facing the same issue. I cannot figure out how to add a GitHub dependency.
Is it really supported?

I've tried the following:
lightbasenl/lightbot.js
lightbasenl/lightbot.js#master
https://github.com/lightbasenl/lightbot.js
https://github.com/lightbasenl/lightbot.js#master

None of them worked, unfortunately :(

Here's the example I'm working on: https://codesandbox.io/s/z6j7954jnx

It's possible to install GitHub dependencies. Keep in mind that we just serve the source files of the GitHub repo (like npm and yarn), we do run the postinstall script, so if you want to build some files you can put it in that step.

Regarding esnet/react-timeseries-charts#fix-355, I just found out that our caching system doesn't support # in the paths, that's why it's showing those errors. I can take a look at that.

Here's the example I'm working on: codesandbox.io/s/z6j7954jnx

I get a different error in that sandbox, about missing regeneratorRuntime. ~I think that's because babel-polyfill is not required in the project. Going to double check that.~

Never mind, that's an issue on our side. Going to take a look!

Committed a fix on master for the regeneratorRuntime error.

A workaround is to use tar.gz URL:

"react-timeseries-charts": "https://github.com/esnet/react-timeseries-charts/archive/fix-355.tar.gz"

Using the tar.gz URL doesn't seem to work anymore. Anyone figured out a working solution for pulling in a dependency from a Github branch?

This issue is stale because it has been open many days with no activity. It will be closed soon unless the stale label is removed or a comment is made.

This issue has been automatically closed because there wasn't any activity after the previous notice or the stale label wasn't removed.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

oliversturm picture oliversturm  ·  3Comments

supersonicclay picture supersonicclay  ·  3Comments

eckmLJE picture eckmLJE  ·  3Comments

waruyama picture waruyama  ·  3Comments

alansouzati picture alansouzati  ·  3Comments