Skip to content

Mark 4 more visual diff tests as flaky#16214

Merged
danielrozenberg merged 3 commits intoampproject:masterfrom
danielrozenberg:more-visual-diff-flakes
Jun 21, 2018
Merged

Mark 4 more visual diff tests as flaky#16214
danielrozenberg merged 3 commits intoampproject:masterfrom
danielrozenberg:more-visual-diff-flakes

Conversation

@danielrozenberg
Copy link
Copy Markdown
Member

More flakes 😢

Related to #16133

@danielrozenberg danielrozenberg changed the title Mark 2 more visual diff tests as flaky Mark 3 more visual diff tests as flaky Jun 21, 2018
@danielrozenberg danielrozenberg changed the title Mark 3 more visual diff tests as flaky Mark 4 more visual diff tests as flaky Jun 21, 2018
@danielrozenberg danielrozenberg merged commit 867e903 into ampproject:master Jun 21, 2018
@danielrozenberg danielrozenberg deleted the more-visual-diff-flakes branch June 21, 2018 18:26
aghassemi added a commit that referenced this pull request Jun 21, 2018
erwinmombay pushed a commit that referenced this pull request Jun 21, 2018
* Revert "Mark 4 more visual diff tests as flaky (#16214)"

This reverts commit 867e903.

* Revert "Update dependencies for default 🌴 (#16195)"

This reverts commit 34d1977.
alabiaga added a commit that referenced this pull request Jun 21, 2018
alabiaga added a commit that referenced this pull request Jun 22, 2018
…6227)

* Revert "Revert "Update dependencies for default 🌴 (#16221)"

This reverts commit f8da39e.

* Revert "Mark 4 more visual diff tests as flaky (#16214)"

This reverts commit 867e903.

* Revert "Update dependencies for default 🌴 (#16195)"

This reverts commit 34d1977.

* Revert "Fix selector (#16197)"

This reverts commit f741868.

* Revert "🐛 Position offscreen amp-story-pages using viewport widths (#16092)"

This reverts commit fa85965.

* Revert "Use performance.now over Date.now in performance metrics (#16119)"

This reverts commit 0d2b567.

* fix lint errors

* fix lint error in constructor

* Update performance-impl.js

fix more lint errors

* Update url-replacements-impl.js

more lint..

* Update test-amp-access.js

fix test error.

* Update test-amp-access-source.js

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants