Move require statements to the beginning

Bluespess / bluespess

MIT License

7 stars 10 forks source link

Open Kinrany opened 5 years ago

Kinrany commented 5 years ago

Move all the require statements that don't have to be dynamic to the beginning of the files

Kinrany commented 5 years ago

Wait, client/lib/lighting.js depends on client/index.js, so that'll probably break something.

monster860 commented 5 years ago

yeah that too

Kinrany commented 5 years ago

I checked the cyclic dependencies with madge, there shouldn't be any now. Well, other than those fixed by do_resolve.

Kinrany commented 5 years ago

Ahh, no, just using mob/index.js won't work because everything is imported as mob.js, not just mob :s

Kinrany commented 5 years ago

Should work now. How do you test this stuff?