Skip to content
This repository was archived by the owner on Jul 29, 2024. It is now read-only.

feat(hybrid): set ng12hybrid flag in the config #3452

Merged
merged 1 commit into from
Oct 26, 2016

Conversation

sjelin
Copy link
Contributor

@sjelin sjelin commented Aug 5, 2016

Response to @juliemr's comments on 71532f0

* @param {string=} opt_baseUrl A URL to prepend to relative gets.
* @param {boolean=} opt_untrackOutstandingTimeouts Whether Browser should
* @param {string=} baseUrl A URL to prepend to relative gets.
* @param {string=} rootElement The
Copy link
Member

Choose a reason for hiding this comment

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

finish line.

@sjelin
Copy link
Contributor Author

sjelin commented Oct 26, 2016

All comments addressed

* @param {boolean=} opt_untrackOutstandingTimeouts Whether Browser should
* @param {string=} baseUrl A URL to prepend to relative gets.
* @param {string=} rootElement The css selector for the element which is the
* root of the Angular app
Copy link
Member

Choose a reason for hiding this comment

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

Nit: Please end sentences with . to be consistent with other comments.

@juliemr
Copy link
Member

juliemr commented Oct 26, 2016

After nit, LGTM and please merge.

@cnishina cnishina merged commit 7083426 into angular:master Oct 26, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants