-
Notifications
You must be signed in to change notification settings - Fork 20
Conversation
…-e2e-tests into add/classic-editor-comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes look good, although I see that newly added test is failing in Jetpack build job.
@JavonDavis This space caused a problem with tests execution. I fixed it and ran tests. Failed ones seem not related to this change. Is it good to go? |
My bad I actually resolved this through Github's online interface to test out that... Yep good to go! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Thanks folks! |
Changes in #1729 caused that comments test is not passing for jetpack sites. I've removed
@jetpack
tag fromCommenting and replying to newly created post in Gutenberg Editor:
and put back testCommenting and replying to newly created post:
.To test:
Make sure that
wp-comments-spec.js
is passing against jetpack sites.