Skip to content

Conversation

jelbourn
Copy link
Member

Also adds lint check to enforce its presence

Also adds lint check to enforce its presence
@googlebot googlebot added the cla: yes PR author has agreed to Google's Contributor License Agreement label Sep 15, 2017
Copy link
Member

@crisbeto crisbeto left a comment

Choose a reason for hiding this comment

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

LGTM

@crisbeto crisbeto added pr: lgtm action: merge The PR is ready for merge by the caretaker and removed pr: needs review labels Sep 15, 2017
@jelbourn jelbourn added P1 Impacts a large percentage of users; if a workaround exists it is partial or overly painful merge: caretaker note Alert the caretaker performing the merge to check the PR for an out of normal action needed or note labels Sep 15, 2017
@jelbourn
Copy link
Member Author

Caretaker note: let me know if this causes any screenshot diffs

"encapsulation": "\\.None$",
"moduleId": "^module\\.id$"
"moduleId": "^module\\.id$",
"preserveWhitespaces": "false$"
Copy link
Contributor

Choose a reason for hiding this comment

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

super nit: is there any reason not to make it "^false$"?

Copy link
Member Author

Choose a reason for hiding this comment

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

Nah, but it's really the same either way

Copy link
Contributor

@tinayuangao tinayuangao left a comment

Choose a reason for hiding this comment

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

LGTM

@jelbourn jelbourn merged commit 2b0315d into angular:master Sep 20, 2017
@jelbourn jelbourn deleted the preserve-whitespace branch April 2, 2018 22:30
@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 8, 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 PR author has agreed to Google's Contributor License Agreement merge: caretaker note Alert the caretaker performing the merge to check the PR for an out of normal action needed or note P1 Impacts a large percentage of users; if a workaround exists it is partial or overly painful

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants