Skip to content

refactor(common): post-review image directive cleanup#47170

Closed
pkozlowski-opensource wants to merge 1 commit intoangular:mainfrom
pkozlowski-opensource:image_post_merge_review
Closed

refactor(common): post-review image directive cleanup#47170
pkozlowski-opensource wants to merge 1 commit intoangular:mainfrom
pkozlowski-opensource:image_post_merge_review

Conversation

@pkozlowski-opensource
Copy link
Member

Minor cleanups based on the review comments that were
not addressed after the initial review.

Based on #47082 (review)

@pkozlowski-opensource pkozlowski-opensource added action: review The PR is still awaiting reviews from at least one requested reviewer area: common Issues related to APIs in the @angular/common package target: minor This PR is targeted for the next minor release labels Aug 17, 2022
@ngbot ngbot bot modified the milestone: Backlog Aug 17, 2022
Copy link
Member

@gkalpak gkalpak left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

FYI, I have left a couple more minor comments in #47082 (review).

@pkozlowski-opensource pkozlowski-opensource marked this pull request as draft August 17, 2022 12:58
@pkozlowski-opensource
Copy link
Member Author

Thnx for the additional review @gkalpak - I've converted this PR to a draft to capture your findings.

Minor cleanups based on the review comments that were
not addressed after the initial review.
@pkozlowski-opensource pkozlowski-opensource marked this pull request as ready for review August 19, 2022 12:12
Copy link
Contributor

@kara kara left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@pkozlowski-opensource pkozlowski-opensource added action: merge The PR is ready for merge by the caretaker and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Aug 22, 2022
@alxhub
Copy link
Member

alxhub commented Aug 22, 2022

This PR was merged into the repository by commit 634e6c7.

@alxhub alxhub closed this in 634e6c7 Aug 22, 2022
pkozlowski-opensource added a commit to pkozlowski-opensource/angular that referenced this pull request Aug 23, 2022
Minor cleanups based on the review comments that were
not addressed after the initial review.

PR Close angular#47170
alxhub pushed a commit that referenced this pull request Aug 23, 2022
Minor cleanups based on the review comments that were
not addressed after the initial review.

PR Close #47170

PR Close #47232
@angular-automatic-lock-bot
Copy link

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.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Sep 22, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: merge The PR is ready for merge by the caretaker area: common Issues related to APIs in the @angular/common package target: minor This PR is targeted for the next minor release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants