You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
refactor: move angular2/lifecycle into angular2/core #4028
3d38ec8 introduces a new module for angular2/lifecycle_hooks, but I think this should be exported from angular2/core since our current approach is for core to have all of the basic building blocks of an angular application.
Task:
Move modules/angular2/lifecycle_hooks.ts to modules/angular2/src/core/lifecycle_hooks.ts
Re-export all lifecycle_hooks exports from modules/angular2/core.ts
@vicb I landed a big PR for #3713 right after this merged in, which reduced the amount of import targets for angular2. Does this change make sense, or is there some use case I'm not thinking about?
I've only moved the hooks in a separate file for doc purpose (ie to have all of them documented in one place and be able to link to the module from directives inline doc).
Activity
vicb commentedon Sep 8, 2015
I've only moved the hooks in a separate file for doc purpose (ie to have all of them documented in one place and be able to link to the module from directives inline doc).
If moving them to core still allows that, I'm +1
tyleranton commentedon Sep 17, 2015
Is this already being worked on? I'd like to start contributing to this project.
jeffbcross commentedon Sep 17, 2015
@tyleranton please take it!
tyleranton commentedon Sep 17, 2015
@jeffbcross awesome, will do!
tyleranton commentedon Sep 18, 2015
@jeffbcross There's an existing lifecycle folder in
src/coreShould it go in
modules/angular2/src/core/lifecycle/lifecycle_hooks.tsinstead of
modules/angular2/src/core/lifecycle_hooks.tsjeffbcross commentedon Sep 23, 2015
@tyleranton I would just move it into
src/core.tyleranton commentedon Sep 24, 2015
@jeffbcross Cool, got it done. Last question.
When re-exporting all
lifecycle_hooksexports, should it be done like so:Or is there a way to specify exporting only
lifecycle_hooksfromangular2/core?2 remaining items
mhevery commentedon Oct 27, 2015
This should be in
angular2/core/contractsper #4834vsavkin commentedon Nov 24, 2015
Fixed here 125fa38
angular-automatic-lock-bot commentedon Sep 7, 2019
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.