Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix error "Cannot find type definition file for fs-capacitor" during bit-build #6573

Merged
merged 3 commits into from
Oct 20, 2022

Conversation

davidfirst
Copy link
Member

No description provided.

@davidfirst davidfirst requested a review from zkochan October 20, 2022 15:16
@zkochan
Copy link
Member

zkochan commented Oct 20, 2022

No changes to the lockfile?

@davidfirst
Copy link
Member Author

No changes to the lockfile?

Lots of changes, but I never commit them, I let the CI handle it. Is it better to have it as part of the commit?

@davidfirst davidfirst enabled auto-merge (squash) October 20, 2022 15:23
@zkochan
Copy link
Member

zkochan commented Oct 20, 2022

It is better because when the lockfile is up-to-date then installation is faster. Also, you see what changes to the deps happened.

@davidfirst
Copy link
Member Author

davidfirst commented Oct 20, 2022

oh ok, makes sense. I added the file.

@davidfirst
Copy link
Member Author

@zkochan , I removed the file :)
Turns out that when I generate the file locally, it's written with the new registry, but circleCI works with the old registry. that's why setup-harmony failed.

@davidfirst davidfirst merged commit 8eeece8 into master Oct 20, 2022
@davidfirst davidfirst deleted the fix-fs-capacitor-types branch October 20, 2022 17:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants