Skip to content

refactor(compiler): Add a new helper method getOwningNgModule.#47166

Closed
dylhunn wants to merge 1 commit intoangular:mainfrom
dylhunn:compiler-apis-1
Closed

refactor(compiler): Add a new helper method getOwningNgModule.#47166
dylhunn wants to merge 1 commit intoangular:mainfrom
dylhunn:compiler-apis-1

Conversation

@dylhunn
Copy link
Contributor

@dylhunn dylhunn commented Aug 17, 2022

This helper accepts a class for an Angular trait, and returns the NgModule which owns that trait. This will be useful for the language service import project, which needs to edit import arrays on the module.

@dylhunn dylhunn added state: WIP target: patch This PR is targeted for the next patch release area: compiler Issues related to `ngc`, Angular's template compiler labels Aug 17, 2022
@ngbot ngbot bot modified the milestone: Backlog Aug 17, 2022
@dylhunn dylhunn force-pushed the compiler-apis-1 branch 3 times, most recently from b101e6a to 4fc9209 Compare August 18, 2022 20:53
@dylhunn dylhunn changed the title feat(compiler): Add new helper APIs getPrimaryAngularDecorator and getOwningNgModule feat(compiler): Add a new helper method getOwningNgModule(). Aug 18, 2022
@dylhunn dylhunn added action: review The PR is still awaiting reviews from at least one requested reviewer and removed state: WIP labels Aug 18, 2022
@dylhunn dylhunn marked this pull request as ready for review August 18, 2022 20:54
@dylhunn dylhunn requested a review from alxhub August 18, 2022 20:55
@pullapprove pullapprove bot requested a review from AndrewKushnir August 18, 2022 20:55
@dylhunn dylhunn requested review from JoostK and removed request for AndrewKushnir August 18, 2022 20:55
@dylhunn dylhunn changed the title feat(compiler): Add a new helper method getOwningNgModule(). feat(compiler): Add a new helper method getOwningNgModule. Aug 18, 2022
@dylhunn dylhunn force-pushed the compiler-apis-1 branch 4 times, most recently from 244a2b1 to 3df60ae Compare August 18, 2022 21:35
Copy link
Member

@alxhub alxhub left a comment

Choose a reason for hiding this comment

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

As with the others, this shouldn't be a feat as it's not a public-facing feature of Angular.

@dylhunn dylhunn changed the title feat(compiler): Add a new helper method getOwningNgModule. refactor(compiler): Add a new helper method getOwningNgModule. Aug 18, 2022
@dylhunn dylhunn 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 action: merge The PR is ready for merge by the caretaker labels Aug 18, 2022
@dylhunn
Copy link
Contributor Author

dylhunn commented Aug 19, 2022

merge-assistance: merge #47180 first; this PR is rebased on that PR

Rebased.

@dylhunn dylhunn added the merge: caretaker note Alert the caretaker performing the merge to check the PR for an out of normal action needed or note label Aug 19, 2022
@ngbot ngbot bot added the action: merge The PR is ready for merge by the caretaker label Aug 19, 2022
@dylhunn dylhunn removed the request for review from JoostK August 19, 2022 02:57
@alxhub alxhub added action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews and removed action: merge The PR is ready for merge by the caretaker merge: caretaker note Alert the caretaker performing the merge to check the PR for an out of normal action needed or note labels Aug 22, 2022
This helper accepts a class for an Angular trait, and returns the NgModule which owns that trait. This will be useful for the language service import project, which needs to edit import arrays on the module.
@dylhunn dylhunn added action: merge The PR is ready for merge by the caretaker and removed action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews labels Aug 22, 2022
@alxhub
Copy link
Member

alxhub commented Aug 22, 2022

This PR was merged into the repository by commit edf7f67.

@alxhub alxhub closed this in edf7f67 Aug 22, 2022
alxhub pushed a commit that referenced this pull request Aug 22, 2022
)

This helper accepts a class for an Angular trait, and returns the NgModule which owns that trait. This will be useful for the language service import project, which needs to edit import arrays on the module.

PR Close #47166
alxhub pushed a commit that referenced this pull request Aug 22, 2022
)

This helper accepts a class for an Angular trait, and returns the NgModule which owns that trait. This will be useful for the language service import project, which needs to edit import arrays on the module.

PR Close #47166
@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
@dylhunn dylhunn deleted the compiler-apis-1 branch October 17, 2022 13:38
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: compiler Issues related to `ngc`, Angular's template compiler target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants