PRs: Remove markdown-links-check from required checks

[snazy]

The "Check Markdown links" workflow is known to produce false failures, leading to #3097.

This change is intended to unblock PRs due to these false failures.

[dimas-b]

+1

Markdown checks are very flaky and have been hindering PRs progress.

Recent example: https://github.com/apache/polaris/actions/runs/19534001072/job/55941993028?pr=2802

ERROR: 1 dead links found!
[✖] https://medium.com/@jitenderkmr/demystifying-snowflake-ids-a-unique-identifier-in-distributed-computing-72796a827c9d → Status: 403

^ that link works fine in a browser.

[binarycat0]

Will it help if extend set of ignorePatterns in .github/workflows/check-md-link-config.json?

{
  "ignorePatterns": [
    {
      "pattern": "https://medium.com/.*"
    },
    {
      "pattern": "^https?://localhost.*"
    },
    {
      "_comment": "mvnrepository blocks requests originating from GitHub Actions",
      "pattern": "https://mvnrepository.com/artifact/org.apache.iceberg/iceberg-spark"
    }
  ]
}

[snazy]

Will it help if extend set of ignorePatterns in .github/workflows/check-md-link-config.json?

{
  "ignorePatterns": [
    {
      "pattern": "https://medium.com/.*"
    },
    {
      "pattern": "^https?://localhost.*"
    },
    {
      "_comment": "mvnrepository blocks requests originating from GitHub Actions",
      "pattern": "https://mvnrepository.com/artifact/org.apache.iceberg/iceberg-spark"
    }
  ]
}

Maybe? ;)
Mind propsing on the dev-ML as a follow-up?

[binarycat-dremio]

LGTM

[pingtimeout]

Approving to unblock CI

A better long-term solution might be to replace the trigger by a manual trigger (like the release workflows). That way we would still be able to check for dead links but we would not use our request quota unless we want it.