fix(ci): Skip docker dry-run for pull requests

[zharinov]

Changes

The build-docker is the disproportionally longest job which is unlikely to break when the non-Docker functionality is being changed in the PR.

This PR delays dry-run step to later stage (e.g. merge queue), which saves one minute for ordinary PRs check time and makes jobs timing uniform.

Context

Documentation (please check one with an [x])

• I have updated the documentation, or
• No documentation update is required

How I've tested my work (please select one)

I have verified these changes via:

• Code inspection only, or
• Newly added/modified unit tests, or
• No unit tests but ran on a real repository, or
• Both unit tests + ran on a real repository

[zharinov]

It's still the longest check. Maybe we could skip it completely in PRs? It would make PR checks finish twice faster.

[viceice]

remember this recently catched null pointer errors which were not detected by unit tests.

[zharinov]

remember this recently catched null pointer errors which were not detected by unit tests.

I don't know about it, but maybe still it's okay to catch things like that at the merge queue stage, in exchange for faster CI for more typical PRs

[renovate-release]

🎉 This PR is included in version 40.48.7 🎉

The release is available on:

• GitHub release
• npm package (@latest dist-tag)
• 40.48.7

Your semantic-release bot 📦🚀