Upgrade checkout and upload-artifacts actions

[Margen67]

The default branch of checkout and upload-artifact was renamed to main years ago:

• https://github.com/actions/checkout/tree/master
• https://github.com/actions/upload-artifact/tree/master

upload-artifacts >v1 doesn't error by default if artifacts aren't found: https://github.com/actions/upload-artifact/?tab=readme-ov-file#customization-if-no-files-are-found

checkout >v1 doesn't fetch the entire commit history by default: https://github.com/actions/checkout?tab=readme-ov-file#checkout-v4

[rb6502]

I don't quite understand the problem here. MAME's primary branch is still called "master" so no changes should be necessary.

[Margen67]

I don't quite understand the problem here. MAME's primary branch is still called "master" so no changes should be necessary.

The master branch of these actions is outdated and no longer used. If you're specifying the master branch I assume you're trying to get the latest version of them.
https://docs.github.com/en/actions/writing-workflows/workflow-syntax-for-github-actions#jobsjob_idstepsuses

[holub]

If you rename workflow to main, that's not magically change our masters and slaves to main.
As long as default branch master - workflow should respect this.

[startaq]

The PR is fine. This updates the actions we use inside the workflow file. It has nothing to do with the naming of our branches.

[cuavas]

What does git describe report when you do this? Does this mean the artifact binaries report “unknown” in place of a git revision now?

[cuavas]

Never mind, I just checked, and apparently it can still work out how many revisions to the last tag, etc.