Skip to content
This repository has been archived by the owner on Feb 18, 2021. It is now read-only.

Put back classic editor comment test #1738

Merged
merged 4 commits into from
Dec 27, 2018
Merged

Conversation

Stojdza
Copy link
Contributor

@Stojdza Stojdza commented Dec 26, 2018

Changes in #1729 caused that comments test is not passing for jetpack sites. I've removed @jetpack tag from Commenting and replying to newly created post in Gutenberg Editor: and put back test Commenting and replying to newly created post:.

To test:
Make sure that wp-comments-spec.js is passing against jetpack sites.

@Stojdza Stojdza self-assigned this Dec 26, 2018
@Stojdza Stojdza requested a review from a team December 26, 2018 13:44
Copy link
Contributor

@brbrr brbrr left a 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
Copy link
Contributor

@Stojdza #1741 was merged and caused a conflict, resolved this and will merge once everything checks out!

@Stojdza
Copy link
Contributor Author

Stojdza commented Dec 27, 2018

@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?

@JavonDavis
Copy link
Contributor

@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!

Copy link
Contributor

@JavonDavis JavonDavis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Stojdza Stojdza merged commit 398724b into master Dec 27, 2018
@Stojdza Stojdza deleted the add/classic-editor-comment branch December 27, 2018 15:56
@brbrr
Copy link
Contributor

brbrr commented Dec 27, 2018

Thanks folks!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
[Status] Needs Jetpack e2e Testing Run full Jetpack e2e suite
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants