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

Replace bootstrap with install in scripts/workflows #6090

Merged
merged 1 commit into from
Jun 14, 2023

Conversation

haltman-at
Copy link
Contributor

Since yarn bootstrap seems to be causing such problems, and I can't figure out the root cause, here's a simple workaround... replacing yarn bootstrap with yarn install. There used to be a difference between these, but due to every package using prepare now, there no longer is. Hopefully this is all OK and solves the problem?

Copy link
Contributor

@cliffoo cliffoo left a comment

Choose a reason for hiding this comment

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

We should probably also update readme (under ### Development) and maybe remove the bootstrap command in package.json?

@haltman-at
Copy link
Contributor Author

We should probably also update readme (under ### Development) and maybe remove the bootstrap command in package.json?

Hm, possibly -- right now the problem is only occurring on CI. @gnidan, any opinion on all this? I'm going to just merge this so that CI can be less of a disaster, and we can figure this out after I figure...

@haltman-at haltman-at merged commit 4d01492 into develop Jun 14, 2023
@haltman-at haltman-at deleted the bootstrap-no-install-yes branch June 14, 2023 16:42
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants