X Tutup
Skip to content

typings: add ability to specify reference paths and ambient module name#3544

Closed
jeffbcross wants to merge 2 commits intoangular:masterfrom
jeffbcross:typing-references
Closed

typings: add ability to specify reference paths and ambient module name#3544
jeffbcross wants to merge 2 commits intoangular:masterfrom
jeffbcross:typing-references

Conversation

@jeffbcross
Copy link
Copy Markdown
Contributor

No description provided.

@jeffbcross jeffbcross added feature Label used to distinguish feature request from other issues area: build & ci Related the build and CI infrastructure of the project action: review The PR is still awaiting reviews from at least one requested reviewer hotlist: typescriptification labels Aug 8, 2015
@jeffbcross jeffbcross added this to the alpha-35 milestone Aug 8, 2015
@jeffbcross
Copy link
Copy Markdown
Contributor Author

FYI @btford breaking change regarding router. This PR changes the global object name of the router to ngRouter instead of ng.router in angular.sfx.dev.js. Discussion in #3458

@jeffbcross jeffbcross force-pushed the typing-references branch 2 times, most recently from 50be427 to c526da2 Compare August 8, 2015 05:14
@alexeagle alexeagle added pr_state: LGTM and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Aug 10, 2015
@alexeagle alexeagle assigned jeffbcross and unassigned alexeagle Aug 10, 2015
Without this feature, each bundle would have to create its own typings
template to include references to dependent typings. Now, the references
can be declared in JS, along with other meta information about the 
bundle typings.


Closes angular#3540
Allows declaring a bundle's namespace in generated typings file,
which should correspond to the global object representing the module
inside its bundle.

BREAKING CHANGE
    The router was previously exported as ng.router in the 
    angular.sfx.dev.js bundle, but now it is exported as ngRouter.

Closes angular#2948
@jeffbcross jeffbcross added the action: merge The PR is ready for merge by the caretaker label Aug 10, 2015
@mary-poppins
Copy link
Copy Markdown

User @jeffbcross does not have PR merging privlidges.

@mary-poppins mary-poppins removed the action: merge The PR is ready for merge by the caretaker label Aug 10, 2015
@jeffbcross
Copy link
Copy Markdown
Contributor Author

Whatever, @mary-poppins

goderbauer pushed a commit to goderbauer/angular that referenced this pull request Aug 15, 2015
Allows declaring a bundle's namespace in generated typings file,
which should correspond to the global object representing the module
inside its bundle.

BREAKING CHANGE
    The router was previously exported as ng.router in the 
    angular.sfx.dev.js bundle, but now it is exported as ngRouter.

Closes angular#2948

Closes angular#3544
@angular-automatic-lock-bot
Copy link
Copy Markdown

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 6, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area: build & ci Related the build and CI infrastructure of the project cla: yes feature Label used to distinguish feature request from other issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

X Tutup