Remove ts-node dependency

pjonsson commented 5 days ago

What this PR does

There are no in-tree uses of ts-node, so make it a dev-dependency.

This also uncovered what seems to be an unused generateModelInterfaces.js, so remove that.

Test me

I believe this is tested by CI.

Checklist

[ ] There are unit tests to verify my changes are correct or unit tests aren't applicable (if so, write quick reason why unit tests don't exist)
[ ] I've updated relevant documentation in doc/.
[ ] I've updated CHANGES.md with what I changed.
[ ] I've provided instructions in the PR description on how to test this PR.

na9da commented 4 days ago

@pjonsson - thanks for the change.

The only remaining use of ts-node is in https://github.com/TerriaJS/terriajs/blob/cf60c17e1643a4dfc08b840b448510108a71a92e/.vscode/launch.json#L18 which i'm not sure anyone is using.

So could we remove it entirely?

pjonsson commented 4 days ago

@na9da I don't know anything about VSCode/the environment the team uses. I don't use VSCode so I'm happy to remove it, so if you want me to remove it instead I'll do that.

na9da commented 4 days ago

Yep, I think we can remove it.

pjonsson commented 4 days ago

Ok, updated PR to remove it instead, and put a note in CHANGES about it being removed.

TerriaJS / terriajs