Skip to content

Fix for missing git commit-based dependency issue#6285

Closed
kumavis wants to merge 13 commits intodevelopfrom
deps-fix
Closed

Fix for missing git commit-based dependency issue#6285
kumavis wants to merge 13 commits intodevelopfrom
deps-fix

Conversation

@kumavis
Copy link
Copy Markdown
Member

@kumavis kumavis commented Mar 12, 2019

Could not install from "node_modules/eth-tx-summary/node_modules/eth-block-tracker/async-eventemitter@github:ahultgren/async-eventemitter#fa06e39e56786ba541c180061dbf2c0a5bbf951c" as it does not contain a package.json file.

@kumavis
Copy link
Copy Markdown
Member Author

kumavis commented Mar 12, 2019

the original error can lead to this next error

npm ERR! path /home/circleci/project/node_modules/ganache-core/node_modules/web3-providers-ws/node_modules/websocket
npm ERR! code EISGIT
npm ERR! git /home/circleci/project/node_modules/ganache-core/node_modules/web3-providers-ws/node_modules/websocket: Appears to be a git repo or submodule.
npm ERR! git     /home/circleci/project/node_modules/ganache-core/node_modules/web3-providers-ws/node_modules/websocket
npm ERR! git Refusing to remove it. Update manually,
npm ERR! git or move it out of the way first.

@kumavis
Copy link
Copy Markdown
Member Author

kumavis commented Mar 12, 2019

cache was bad so i added an environment var to CircleCI so we can manually change that to reset the cache without pushing commits

seems like a hack but its the official reccomendation https://support.circleci.com/hc/en-us/articles/115015426888-Clear-project-cache

@kumavis kumavis requested review from danjm and whymarrh as code owners March 12, 2019 08:09
@kumavis
Copy link
Copy Markdown
Member Author

kumavis commented Mar 12, 2019

woof this spiraled out of control a bit

@Azier777

This comment has been minimized.

Sign in to view
@kumavis
Copy link
Copy Markdown
Member Author

kumavis commented Mar 12, 2019

closing in favor of #6286

@kumavis kumavis closed this Mar 12, 2019
@kumavis kumavis deleted the deps-fix branch March 12, 2019 10:41
@kumavis kumavis mentioned this pull request Mar 12, 2019
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@danjm danjm Awaiting requested review from danjm

@whymarrh whymarrh Awaiting requested review from whymarrh

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@kumavis @Azier777