search

VirtualWhirlwind / vscode-povray2

A Visual Studio Code extension to allow you to edit Persistence of Vision Ray Tracer (POV-Ray) Scene files and render them using the integrated terminal
MIT License
6 stars 3 forks source link

Reorganize code to break up the large extension.ts file #5

Open kennebel opened 2 years ago

kennebel commented 2 years ago

See if we can break up the code to functional areas where it makes sense to focus the extension.ts file instead of doing everything.

bentorkington commented 2 years ago

I recommend as part of this:

  1. Updating the deps - some of which are obsolete or unavailable
  2. Settling on yarn vs npm: Some actions in the package.json have been set up to use yarn, but this complains because there is a package-lock.json file in the repo
  3. Not including the .vsix installers in the repo, as a repo should generally not include its build output
kennebel commented 2 years ago

Thanks for the help! I did a "hello world" in typescript a few years ago, but that is as far as I went at the time. TS/JS is not part of my normal coding routine, so there is a lot I don't know about. When i forked this project, i fixed the errors that popped up that was preventing the project from running, but there are still many mysteries regarding TS and VSCode Extensions.

As far as including the vsix, I was following the example from the previous author having it included, didn't know if that was "the style" these days. :)