search

thefrontside / simulacrum

A simulation platform for use during testing, during development and for high-fidelity application previews
89 stars 14 forks source link

parcel.3.1 configuration and build failure #138

Closed jbolda closed 2 years ago

jbolda commented 2 years ago

Motivation

Parcel seems to be failing with our configuration on beta.3.1.

Approach

Upgrading and fixing the configuration.

Questions

The build seems to pass just fine on CircleCI, but I am hitting the error locally and in CI on another PR. I think this is a cache error issue?

image

CI failure: https://github.com/thefrontside/simulacrum/pull/122/commits/205fb9140ba3a5a17d5948b9c6ce4adcae999770 https://app.circleci.com/pipelines/github/thefrontside/simulacrum/637/workflows/a4209c11-df88-4d35-9510-0dca29d7a83d/jobs/717

cowboyd commented 2 years ago

There is a fix running around in like 3 branches 🤦🏻‍♂️

Also, we ran across this while extracting the Bigtest UI and authored the correct package.json there https://github.com/thefrontside/bigtest/pull/1021/files#diff-1ae5a5e8c23bd3bc31ea590a9cbe48eb9ad3bbddc0fb5732d08c04a53c8ad51dR6-R16

The key is to create an "app" target in package.json and then have all the build scripts use it

dagda1 commented 2 years ago

There is a fix running around in like 3 branches 🤦🏻‍♂️

There is a fix for this in the add-cypress-integration branch here and the branch is good to go I think.

jbolda commented 2 years ago

Still don't know why it is passing in CircleCI as it should be failing with a configuration error. There must be some cache issue that we aren't busting here. Something to keep in mind, but I don't know that it is worth pursuing.

jbolda commented 2 years ago

We had a few different issues going on that I was conflating. The issue shown in my screenshot is fixed in #139. The issue on CircleCI in the linked branch was solved in that branched. Closing.