website icon indicating copy to clipboard operation
website copied to clipboard

Do we really need Babel?

Open RMcNeely opened this issue 1 year ago • 7 comments

I would need to look closer but do we really need Babel as dependency? We aren't using JSX or any other crazy features. Is it possible to drop this?

RMcNeely avatar Jul 29 '24 21:07 RMcNeely

Possibly not, we are on Node v18.17.1 with NPM 9.6.7 on Ubuntu 22.04.4 on the server if you want to test removing it.

lewisgoddard avatar Jul 30 '24 08:07 lewisgoddard

I tried out removing babel from the files and I'm pretty sure my method was not very clean so might have to fix that but as far as I can tell when I launched it on docker it instantly started the server instead of taking some time and there are no bugs as far as I can see so I might send a PR in a bit but I'm going to test it a bit more first

Lucas-Code27 avatar Jul 09 '25 22:07 Lucas-Code27

ok so I reset it and removed the packages properly and only 2 modules of babel are actually needed as dependencies but after I removed the others I see this message in the get involved page

Image

pretty sure this is the only problem I'm running this through docker on Fedora 42 in firefox and I did not notice the performance improvement this time so it must have been because I removed another module the last time I tested if anyone knows why this happens let me know I think we can run the website without most of babel

Lucas-Code27 avatar Jul 09 '25 22:07 Lucas-Code27

Cool got a PR where it can be tested?

RMcNeely avatar Jul 09 '25 23:07 RMcNeely

sure let me send it up

Lucas-Code27 avatar Jul 09 '25 23:07 Lucas-Code27

Sorry I sent the wrong PR thought this was a different issue let me send up the correct one

Lucas-Code27 avatar Jul 09 '25 23:07 Lucas-Code27

ok I sent the PR

Lucas-Code27 avatar Jul 09 '25 23:07 Lucas-Code27