X Tutup
Skip to content

Ng class cleanup#7202

Closed
mhevery wants to merge 1 commit intoangular:masterfrom
mhevery:ngClassCleanup
Closed

Ng class cleanup#7202
mhevery wants to merge 1 commit intoangular:masterfrom
mhevery:ngClassCleanup

Conversation

@mhevery
Copy link
Copy Markdown
Contributor

@mhevery mhevery commented Feb 20, 2016

Ignore noImplicitAny changes they will be merged with #7030 and are all green. The failure seems to come from the ng_class.ts changes, but I don't see a reason why.

@mhevery mhevery force-pushed the ngClassCleanup branch 4 times, most recently from 63baac1 to 5fbd253 Compare February 20, 2016 20:07
@mhevery
Copy link
Copy Markdown
Contributor Author

mhevery commented Feb 22, 2016

@juliemr / @jelbourn can you help me debug https://travis-ci.org/angular/angular/jobs/110646029#L1275 There is an end 2 end test failing and I am not sure as to why. (Ignore the first SHA, it passes here: https://travis-ci.org/angular/angular/builds/110551804) The only relevant change is ngClass file here: mhevery@5fbd253#diff-2d9cd73c23ec9bc747111f8bf4caed4bL1

@mhevery
Copy link
Copy Markdown
Contributor Author

mhevery commented Feb 22, 2016

@juliemr / @jelbourn never mind, the issue magically fixed itself. :-)

@mhevery
Copy link
Copy Markdown
Contributor Author

mhevery commented Feb 22, 2016

@vsavkin please review.

@mhevery
Copy link
Copy Markdown
Contributor Author

mhevery commented Mar 3, 2016

@vsavkin LGTMed

@mhevery mhevery added pr_state: LGTM action: merge The PR is ready for merge by the caretaker labels Mar 3, 2016
@vikerman
Copy link
Copy Markdown
Contributor

vikerman commented Mar 4, 2016

TAP Green

@mary-poppins
Copy link
Copy Markdown

Merging PR #7202 on behalf of @vikerman to branch presubmit-vikerman-pr-7202.

@mhevery mhevery closed this in 6402d61 Mar 4, 2016
@mhevery mhevery deleted the ngClassCleanup branch June 2, 2017 17:05
@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 11, 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

X Tutup