I addressed all of the feedback and it should be ready to go

Don't forget to run npm run all so it creates the distribution.

I have no experience packaging actions, so just out of curiosity, is the bundling of dist/index.js necessary? It feels like that should be a build/deploy step and not something that's checked into the repo.

#72 was merged. Can you please rebase?

Rebased and version changed

Does src need to be set to anything special? The test failed because it wants to reformat files, so does it need to be in it's own sub-directory and have src set to that?

Looks like a npm run all is missing and we are good to go

I ran it and nothing had changed. I just tried again and still nothing was different, so I'm not sure what to do there.

Can you rebase on main? You might be missing changes which get merged in the CI build

I hadn't run npm install so that's why it was out of date. Should be good now

I hadn't run npm install so that's why it was out of date. Should be good now

Ah right I always forget about this!

I hadn't run npm install so that's why it was out of date. Should be good now

Ah right I always forget about this!

Maybe add it to nom run all?

Thanks!