Support requirements.txt for version-file

[daveisfera]

No description provided.

[eifinger]

Thank you for your contribution!

A few changes and we are good to go.

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

[daveisfera]

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

[daveisfera]

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.

[eifinger]

#72 was merged. Can you please rebase?

[daveisfera]

Rebased and version changed

[daveisfera]

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?

[eifinger]

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

[daveisfera]

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.

[eifinger]

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

[daveisfera]

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

[eifinger]

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!

[daveisfera]

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?

[eifinger]

Thanks!