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

node 18 #709

Merged
merged 13 commits into from
Dec 31, 2023
Merged

node 18 #709

merged 13 commits into from
Dec 31, 2023

Conversation

rom1504
Copy link
Member

@rom1504 rom1504 commented Jun 3, 2023

No description provided.

@rom1504
Copy link
Member Author

rom1504 commented Jun 3, 2023

idk where is that require-self mention

@extremeheat
Copy link
Member

it's from node-minecraft-data https://github.com/PrismarineJS/node-minecraft-data/blob/master/package.json#LL13C17-L13C80

None of the prepare scripts seem to need it so it can probably be removed, but the problem maybe it's not installing the devDeps for some reason

@rom1504
Copy link
Member Author

rom1504 commented Jun 3, 2023

we can simply replace require self by self dep
PrismarineJS/node-minecraft-data#286

@rom1504
Copy link
Member Author

rom1504 commented Aug 5, 2023

I don't understand why it fails with npm ERR! Error: Cannot find module 'json-schema-to-typescript'

@rom1504
Copy link
Member Author

rom1504 commented Dec 31, 2023

so error here is still npm ERR! Error: Cannot find module 'json-schema-to-typescript'

tried various things but nothing helps

any more idea here @extremeheat ?

@extremeheat
Copy link
Member

Should be fixed, not sure what flow was going on previously

@rom1504 rom1504 merged commit 4360b84 into master Dec 31, 2023
4 checks passed
@rom1504 rom1504 deleted the node18 branch December 31, 2023 22:20
@rom1504
Copy link
Member Author

rom1504 commented Dec 31, 2023

Thanks

AlexProgrammerDE pushed a commit to AlexProgrammerDE/minecraft-data that referenced this pull request Jan 1, 2024
* node 18

* Update bedrock-ci.yml

* Update update-helper.yml

* Update ci.yml

* Update bedrock-ci.yml

* Update bedrock-ci.yml

* Update bedrock-ci.yml

* Update bedrock-ci.yml

* Update bedrock-ci.yml order

* Update bedrock-ci.yml

---------

Co-authored-by: extremeheat <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Closed
Development

Successfully merging this pull request may close these issues.

2 participants