CI: Fix Docs Deploy

[HTRamsey]

I guess the output from the build step wasn't working like I had expected.
I think it might be worthwhile to let it upload the build artifact to review locally, so i removed the requirement for that part, but left the check for deployment.

[hamishwillee]

Perhaps one for @DonLakeFlyer to review. This looks OK to me, but since this isn't in /docs path the workflow itself is untested.

Note, we should also update the other workflows to make sure that they ignore changes in the /docs path.

[DonLakeFlyer]

So this basically goes back to the way it was before right? I'm not super familiar with the docs workflow.

[HTRamsey]

It still has the node version change, and uploads the build result artifact, but otherwise yes. We will have to manually run the docs workflow to double check though.

[hamishwillee]

Note, we should also update the other workflows to make sure that they ignore changes in the /docs path.

NOte that again ^^^ - every time I make a docs change it forces a rebuild on everything. That doesn't need to happen.

[HTRamsey]

They should all have

paths-ignore:
      - 'docs/**'