We talked about this on Discord and I underestimated how much effort this would be.
There was a concerning discrepency between the files in the dist directory compared to the ones in the src directory. It also seemed like the macros folder both was and wasn't intended to be included in the dist.
Build Changes
Added steps to the build workflow to run npm run build
Removed dist directory and added it to gitignore. This directory should be built from the /src during CI and distributed, changes should all be in the src directory.
Consolidated Files
Any files missing from /src in /dist were copied into /src.
card.js was removed as it was empty without TS
Typescript removal
I ran each file through Sucrase's demo with the only checkbox active being 'typescript'. Sometimes I took their conversion but most of the time I took the tsc conversion after making some tweaks to the original file.
Testing
After deleting my /dist directory and running npm run build fresh, I linked the /dist directory to a module in an 079 build and opened it up. Everything seemed to be working but I have limited knowledge of all the things Card Support is supposed to accomplish.
We talked about this on Discord and I underestimated how much effort this would be.
There was a concerning discrepency between the files in the
dist
directory compared to the ones in thesrc
directory. It also seemed like themacros
folder both was and wasn't intended to be included in the dist.Build Changes
npm run build
dist
directory and added it to gitignore. This directory should be built from the /src during CI and distributed, changes should all be in the src directory.Consolidated Files
card.js
was removed as it was empty without TSTypescript removal
Testing
After deleting my /dist directory and running
npm run build
fresh, I linked the /dist directory to a module in an 079 build and opened it up. Everything seemed to be working but I have limited knowledge of all the things Card Support is supposed to accomplish.