Skip to content

Conversation

kara
Copy link
Contributor

@kara kara commented Apr 14, 2016

@kara kara added the action: review The PR is still awaiting reviews from at least one requested reviewer label Apr 14, 2016
@kara kara added state: WIP and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Apr 14, 2016
@kara kara added action: review The PR is still awaiting reviews from at least one requested reviewer and removed state: WIP labels Apr 14, 2016
@@ -105,6 +105,9 @@ export function main() {

it('should ignore comments in expressions', () => { checkAction('a //comment', 'a'); });

it('should retain // in quotes',
Copy link
Contributor

Choose a reason for hiding this comment

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

say "string literal"

@vsavkin
Copy link
Contributor

vsavkin commented Apr 14, 2016

Also, rename the second commit. The issue is not url specific.

@vsavkin vsavkin 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: review The PR is still awaiting reviews from at least one requested reviewer labels Apr 14, 2016
@vsavkin vsavkin assigned kara and unassigned vsavkin Apr 14, 2016
@kara kara force-pushed the url branch 2 times, most recently from 4b0b6b9 to ca30d95 Compare April 14, 2016 20:52
@kara kara changed the title fix(parser): only strip comments if not urls fix(parser): only strip comments if not string literals Apr 14, 2016
@kara kara force-pushed the url branch 2 times, most recently from dfde3da to aea83b9 Compare April 14, 2016 21:37
@kara kara added action: review The PR is still awaiting reviews from at least one requested reviewer 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 Apr 14, 2016
@vsavkin vsavkin added pr_state: LGTM and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Apr 14, 2016
@kara kara added the action: merge The PR is ready for merge by the caretaker label Apr 14, 2016
@kara kara changed the title fix(parser): only strip comments if not string literals feat(i18n): add custom placeholder names Apr 14, 2016
@mary-poppins
Copy link

User @robertmesserle does not have PR merging privileges.

@mary-poppins
Copy link

Merging PR #8057 on behalf of @robertmesserle to branch presubmit-robertmesserle-pr-8057.

@mhevery mhevery closed this in bb9fb21 Apr 18, 2016
@kara kara deleted the url branch October 13, 2018 01:10
@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 14, 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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants