refactor(docs-infra): refactors addImageDimensionsImpl

[sonukapoor]

Change addImageDimensionsImpl to reduce nesting.

PR Checklist

Please check if your PR fulfills the following requirements:

• The commit message follows our guidelines: https://github.com/angular/angular/blob/master/CONTRIBUTING.md#commit
• Tests for the changes have been added (for bug fixes / features)
• Docs have been added / updated (for bug fixes / features)

PR Type

What kind of change does this PR introduce?

• Bugfix
• Feature
• Code style update (formatting, local variables)
• Refactoring (no functional changes, no api changes)
• Build related changes
• CI related changes
• Documentation content changes
• angular.io application / infrastructure changes
• Other... Please describe:

Does this PR introduce a breaking change?

• Yes
• No

[petebacondarwin]

LGTM - unfortunately you need to make the body of the commit message longer (>100 chars) to pass the lint check.

[ngbot]

I see that you just added the PR action: merge label, but the following checks are still failing:
     status "ci/circleci: lint" is failing
     missing required labels: PR target: *
     forbidden labels detected: PR action: cleanup
     1 pending code review

If you want your PR to be merged, it has to pass all the CI checks.

If you can't get the PR to a green state due to flakes or broken master, please try rebasing to master and/or restarting the CI job. If that fails and you believe that the issue is not due to your change, please contact the caretaker and ask for help.

[sonukapoor]

LGTM - unfortunately you need to make the body of the commit message longer (>100 chars) to pass the lint check.

I saw this new requirement, but at the same time, I can see PRs that are merged without fulfilling them - Is that normal?

This one for example 77d8afb

[petebacondarwin]

That looks like a special commit created by the release script. So I don't know if it had to go through the same checks.

[sonukapoor]

That looks like a special commit created by the release script. So I don't know if it had to go through the same checks.

Yes, I will try to add some other info to the body

[angular-automatic-lock-bot]

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

_{This action has been performed automatically by a bot.}