Skip to content

Remove wd usage and removes disabled tests#6311

Closed
geriux wants to merge 2 commits into
version-toolkit/gutenberg/rnmobile/appium-2from
test/wd-removal-tests-cleanup
Closed

Remove wd usage and removes disabled tests#6311
geriux wants to merge 2 commits into
version-toolkit/gutenberg/rnmobile/appium-2from
test/wd-removal-tests-cleanup

Conversation

@geriux

@geriux geriux commented Oct 23, 2023

Copy link
Copy Markdown
Contributor

Related PRs:

This PR updates the Gutenberg reference where we remove the remaining usage of the wd library, it also removes a few tests that have been disabled for a long time.

To test CI checks should pass.

PR submission checklist:

  • I have considered adding unit tests where possible.
  • I have considered if this change warrants user-facing release notes more info and have added them to RELEASE-NOTES.txt if necessary.

@geriux geriux added the Testing Anything related to automated tests label Oct 23, 2023
@geriux geriux marked this pull request as ready for review October 23, 2023 12:31
@geriux geriux requested a review from dcalhoun October 23, 2023 12:32

@dcalhoun dcalhoun left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Approved with one note to consider in WordPress/gutenberg#55541 (review).

@geriux

geriux commented Oct 23, 2023

Copy link
Copy Markdown
Contributor Author

Closing since the Gutenberg reference was updated in the base branch.

@geriux geriux closed this Oct 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Testing Anything related to automated tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants