search

Dezudio / dezudio-website

0 stars 0 forks source link

Our Approach Section Styleing #38

Closed mrdirby closed 2 years ago

mrdirby commented 3 years ago

Second attempt at this pull request. Hoping that there are no merge conflicts again.

netlify[bot] commented 3 years ago

❌ Deploy Preview for dezudio failed.

🔨 Explore the source changes: a6931298ecb3adf285b1337d949773bc63ddec39

🔍 Inspect the deploy log: https://app.netlify.com/sites/dezudio/deploys/61606359daade00008b6a9bb

mrdirby commented 3 years ago

@merichar I created a new pull request to merge these changes. It looks like its deploys that are failing and the merge itself doesn't have issues. I'm still a hesitant to move forward with the merge because I don't want to take an action that would take more work to rewind.

A look at the log file shows this error on line 44. 11:28:08 AM: npm WARN read-shrinkwrap This version of npm is compatible with lockfileVersion@1, but package-lock.json was generated for lockfileVersion@2. I'll try to do my best with it! My commits don't change the package-lock.json file.

I don't know if this solution is helpful. When it comes to npm errors I really struggle. https://stackoverflow.com/questions/64813775/is-there-any-way-to-fix-package-lock-json-lockfileversion-so-npm-uses-a-specific

merichar commented 2 years ago

Netlify is confused about a darwin system env. We can override it with a conf, it just hadn't come up yet because I'd been on Linux. Since it's the backup review site, let's merge the pr to keep moving.