latticexyz / mud

MUD is a framework for building autonomous worlds
https://mud.dev
MIT License
718 stars 178 forks source link

consider disabling node warnings for deploy #3128

Open holic opened 1 week ago

holic commented 1 week ago

apparently can use NODE_NO_WARNINGS=1

otherwise its noisy

world-deployer  | (node:458) ExperimentalWarning: Importing JSON modules is an experimental feature. This feature could change at any time
world-deployer  | (Use node --trace-warnings ... to show where the warning was created)
world-deployer  | (node:294) ExperimentalWarning: The Fetch API is an experimental feature. This feature could change at any time
Progress : [###########-----------------------------] 28.57%
Progress : [#################-----------------------] 42.86%
world-deployer  | (node:760) ExperimentalWarning: Importing JSON modules is an experimental feature. This feature could change at any time
world-deployer  | (Use node --trace-warnings ... to show where the warning was created)
world-deployer  | (node:925) ExperimentalWarning: Importing JSON modules is an experimental feature. This feature could change at any time
world-deployer  | (Use node --trace-warnings ... to show where the warning was created)
world-deployer  | (node:760) ExperimentalWarning: The Fetch API is an experimental feature. This feature could change at any time

might be coming from https://github.com/search?q=repo%3Alatticexyz%2Fmud+%22assert+%7B+type%3A+%5C%22json%5C%22+%7D%3B%22&type=code

holic commented 1 week ago

could look into https://github.com/nodejs/node/issues/51347#issuecomment-2111337854 as well, but unclear if we'd miss out on TS types for JSON ABIs if we use this