Update @actions/artifact to v2

[HealthyPear]

This is needed because of the recurrent upstream issue of the dependent action upload-artifact (see actions/upload-artifact#490) which is currently causing showyourwork/showyourwork#526.

@mkenworthy could you test this PR by using this version of the action from your project's workflows?
Same as last time: change the name of the branch, now from main to update-artifact-action

@dfm the node modules bundle is now 120Mb, so ~6 times heavier than before; I think this is already only the production dependencies (I used npm install --omit=dev) and the heaviest dep is typescript which in my understanding is what the new version of the action in subject is using, so I do not know if we can make it smaller now.

Closes #16
Closes showyourwork/showyourwork#526

[HealthyPear]

@mkenworthy actually sorry this branch comes from my fork, so you should change the whole line to use my fork

from

uses: showyourwork/showyourwork-action@v1

to

uses: HealthyPear/showyourwork-action@update-artifact-action

[mkenworthy]

which build files do I need to change this in? I already tried it and it complained about the 'wrong Python version' bug from a few weeks ago...

[HealthyPear]

which build files do I need to change this in?

.github/workflows/build-pull-request.yml

and

.github/workflows/build.yml

I already tried it and it complained about the 'wrong Python version' bug from a few weeks ago...

This was my fault, apologies!
I branched off from my fork's main branch which was not updated from the last merge, so that fix was not taken into account.
I now updated this branch by rebasing it on top of the latest main. Now it should work.

[HealthyPear]

OK I am testing this here, but it seems that their API has been changed, so I'll have to look into it...

[HealthyPear]

OK I think I managed to fix this.

You can check here that in my test project I used first the action from the main branch failing to complete the workflow, while it succeeded when using this branch.

@mkenworthy can you also test it with your project as we discussed above?

@dfm would you like to crosscheck I did things correctly?

[mkenworthy]

Just tried this on my project, and it works! Thank you, it's great to have the compiled PDF.

[HealthyPear]

Great!

@mkenworthy I will wait a few days to see if some other project maintainer wants to add something

When this PR will be merged you should be notified automatically, then remember to change your action specification to the main branch of the official repository.