New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix undefined variable #53

Merged

mprobst merged 1 commit into angular:master from alexander-fenster:patch-1

Jan 19, 2018

Contributor

alexander-fenster commented Jan 19, 2018

Looks like this code is not executed too often but I was lucky enough to get this error.


          fix undefined variable

832f4de

Looks like this code is not executed too often but I was lucky enough to get this error.

googlebot added the cla: yes label

evmar requested a review from mprobst

January 19, 2018 00:06

Contributor

mprobst commented Jan 19, 2018

Whoops, thanks. We don't have good tests, and no tests for the error case, I'm afraid.

mprobst approved these changes

View reviewed changes

mprobst merged commit 19ccfab into angular:master

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels