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

Build Error on Blockbase and Children #6365

Closed
wants to merge 1 commit into from
Closed

Build Error on Blockbase and Children #6365

wants to merge 1 commit into from

Conversation

bobmatyas
Copy link
Contributor

Creating this PR per Slack thread. It isn't actually to be merged.

I'm having trouble building Zoologist. When I npm run build I get:

Error: Can't find stylesheet to import.
  ╷
1 │ @import '../../node_modules/@wordpress/base-styles/breakpoints';
  │         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  ╵
  ../blockbase/sass/base/_breakpoints.scss 1:9  @import
  sass/theme.scss 1:9                           root stylesheet 

I did npm install on Zoologist and on Blockbase. It also looks like the ../../node_modules/@wordpress/base-styles/breakpoints file is present:

1037 Aug 10 15:41 _breakpoints.scss

I get the same error when I try to build Blockbase.

Any help would be appreciated :)

@bobmatyas bobmatyas requested a review from a team August 10, 2022 21:17
@pbking
Copy link
Contributor

pbking commented Aug 11, 2022

I was able to reproduce with node 16.x
Things worked fine for me for 14.x

To reproduce I had to delete my node_modules folder I had in my existing /blockbase folder and re-npm install. A subsequent npm run build resulted in the error noted above.

I believe this is because of our recent shift to a npm workspace (#6200) which resulted in all node modules being stored in the root project rather than individual project /node_modules folder.

@pbking pbking changed the title Build Error on Zoologist Build Error on Blockbase and Children Aug 11, 2022
@madhusudhand
Copy link
Member

@bobmatyas Thanks for the report. #6371 should fix the issue.

@madhusudhand madhusudhand deleted the fix/6085 branch August 11, 2022 09:20
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.

3 participants