X Tutup
Skip to content

fix(dart/transform): Consider of line numbers in inliner_for_test#5285

Closed
kegluneq wants to merge 1 commit intoangular:masterfrom
kegluneq:fix_i5281
Closed

fix(dart/transform): Consider of line numbers in inliner_for_test#5285
kegluneq wants to merge 1 commit intoangular:masterfrom
kegluneq:fix_i5281

Conversation

@kegluneq
Copy link
Copy Markdown

Ensure that line numbers aren't changed by inliner_for_test.

Fixes #5281

@kegluneq kegluneq added type: bug/fix comp: dart-transformer action: review The PR is still awaiting reviews from at least one requested reviewer labels Nov 14, 2015
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

add "Style" to the key for readability: _vStyle0

@jbdeboer
Copy link
Copy Markdown

Thanks for the quick fix. The team and our stack traces appreciate it !

@kegluneq kegluneq force-pushed the fix_i5281 branch 2 times, most recently from fd84d98 to e30c845 Compare November 16, 2015 17:00
Ensure that line numbers aren't changed by inliner_for_test.

Fixes angular#5281
@kegluneq
Copy link
Copy Markdown
Author

Updated the inline value names to _template$num and _style$num, plus updated to provide print order guarantees since the unit tests depend on it.

PTAL

@tbosch tbosch added pr_state: LGTM and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Nov 16, 2015
@kegluneq
Copy link
Copy Markdown
Author

Ready for commit - thanks!
/cc @vsavkin

@kegluneq kegluneq assigned vsavkin and unassigned jbdeboer Nov 16, 2015
@kegluneq kegluneq added the action: merge The PR is ready for merge by the caretaker label Nov 16, 2015
@mary-poppins
Copy link
Copy Markdown

Merging PR #5285 on behalf of @vsavkin to branch presubmit-vsavkin-pr-5285.

@mary-poppins
Copy link
Copy Markdown

Merging PR #5285 on behalf of @vsavkin to branch presubmit-vsavkin-pr-5285.

@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 7, 2019
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 cla: yes type: bug/fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Angular transformer is adding extra lines to Dart files -- the line number in stack traces do not match our source code

6 participants

X Tutup