Skip to content
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.

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

Fix and unskip more tests that were disabled due to lolex upgrade #12525

Merged
merged 1 commit into from Dec 19, 2017
Merged

Fix and unskip more tests that were disabled due to lolex upgrade #12525

merged 1 commit into from Dec 19, 2017

Conversation

rsimha
Copy link
Contributor

@rsimha rsimha commented Dec 19, 2017

Follow up to #12486

@rsimha
Copy link
Contributor Author

rsimha commented Dec 19, 2017

/to @erwinmombay @aghassemi
/cc @lannka @cramforce

@cathyxz
Copy link
Contributor

cathyxz commented Dec 19, 2017

Yay! ty for this, I can take a stab at fixing the amp-sidebar tests.

@rsimha
Copy link
Contributor Author

rsimha commented Dec 19, 2017

@cathyxz Beat you to it. Already fixed in #12523 :)

@cathyxz
Copy link
Contributor

cathyxz commented Dec 19, 2017

Oh lol. Yay thanks!

@rsimha rsimha merged commit b1f103f into ampproject:master Dec 19, 2017
@rsimha rsimha deleted the 2017-12-19-MoreLolex branch December 19, 2017 17:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants