-
Notifications
You must be signed in to change notification settings - Fork 92
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 build after merge of Node 18 #1844
Conversation
@@ -7140,25 +7140,26 @@ fsevents@^1.2.7: | |||
bindings: ^1.5.0 | |||
nan: ^2.12.1 | |||
checksum: ae855aa737aaa2f9167e9f70417cf6e45a5cd11918e1fee9923709a0149be52416d765433b4aeff56c789b1152e718cd1b13ddec6043b78cdda68260d86383c1 | |||
conditions: os=darwin |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is really interesting - not sure exactly what it means when we try to build on a non-macOS system.
So this is now red because of the node-sass thing in #1834 , but that one is failing on this issue, so they are co-dependent. Thinking we should merge this one first, @MelsHyrule WDYT? |
@MelsHyrule I'm trying something fancy and merging #1834 into this PR branch 😮 |
23d30ba
to
7d6935e
Compare
This commit is similar to ManageIQ/manageiq-ui-classic#8747
Ok, I merged #1834 and rebased this PR. I expect this to now go green. |
Tested and LGTM |
@MelsHyrule Please review - I think this should fix master now that we are on node 18.
If this works I'll send out updates across the board.