ci(plugins-ci): require quality-checks to pass

[Fdawgs]

always() overrides the results of the needs array, so even though linting failed in fastify/busboy#220, the automerge still occurred. always() is used so fastify-dependency-integration can be skipped if not enabled.

Spotted thanks to @Tony133 in fastify/busboy#221

Checklist

• run npm run test && npm run benchmark --if-present
• tests and/or benchmarks are included
• documentation is changed or added
• commit message and code follows the Developer's Certification of Origin
  and the Code of conduct

[Copilot]

Pull request overview

This PR adjusts the plugins-ci workflow’s Dependabot automerge gating to prevent automerges when dependent jobs fail by removing always() from the automerge job condition.

Changes:

• Remove always() from the automerge job’s if: condition in .github/workflows/plugins-ci.yml.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Tony133]

@Fdawgs I think need.quality-check.result == 'success' is not enough, dependabot continues to bypass even if the tests have failed by doing only 2 or 3 checks (see screenshot and here: fastify/fastify-type-provider-json-schema-to-ts#147)

[Fdawgs]

@Tony133 this change hasn't been released or deployed anywhere yet is why. 😇

[Tony133]

@Tony133 this change hasn't been released or deployed anywhere yet is why. 😇

ah ok 😅 👍🏻, it's better to do a release, I'm blocking the upgrade to typescript v7.x because it breaks the lint, it's not compatible with eslint v9 and neostandard 😆