Download shouldn't break if if_no_artifact_found is set to !fail

[mkarasik]

I have a multiple artifacts in my run, and one of them is expired. However the download loop stops regardless of the flag.

https://github.com/dawidd6/action-download-artifact/blob/master/main.js#L257

    } catch (error) {
      if (error.message.startsWith("Artifact has expired")) {
          return setExitMessage(ifNoArtifactFound, "no downloadable artifacts found (expired)")
      } else {
          throw new Error(error.message)
      }

Warn and ignore should not break the loop.

[mkarasik]

I patched in my branch as below

          } catch (error) {
                if (error.message.startsWith("Artifact has expired")) {
                    setExitMessage(ifNoArtifactFound, "no downloadable artifacts found (expired)")
                    if (ifNoArtifactFound === "fail")
                        return
                    else
                        continue
                } else {
                    throw new Error(error.message)
                }
            }

but it doesn't look like the correct code. I wonder if the artifact status should be checked and logged before the download loop started.
Any comments on this?