Skip to content

docs: Clarify the use of Gutenberg and Gutenberg Mobile repository names#6527

Merged
dcalhoun merged 4 commits into
trunkfrom
docs/clarify-e2e-test-repo-names
Jan 10, 2024
Merged

docs: Clarify the use of Gutenberg and Gutenberg Mobile repository names#6527
dcalhoun merged 4 commits into
trunkfrom
docs/clarify-e2e-test-repo-names

Conversation

@derekblank
Copy link
Copy Markdown
Contributor

Clarifies the references to Gutenberg and Gutenberg Mobile repositories in E2E testing docs.

@dcalhoun in the context of previous changes in 4574af6, please verify if the intent in this PR is correct, and feel free to make and changes or suggestions directly.

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.

Comment thread README.md Outdated
Comment thread README.md Outdated
Copy link
Copy Markdown
Member

@dcalhoun dcalhoun left a comment

Choose a reason for hiding this comment

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

Thank you for proposing changes to improve clarity. I will follow up and move this forward.

Comment thread README.md Outdated
Comment thread README.md Outdated
Comment thread README.md Outdated
Attempt to mitigate confusion between the `gutenberg` and
`gutenberg-mobile` repositories, in addition to the general "Gutenberg"
project and plugin.
It is better to describe the current state, rather than an ideal.
@dcalhoun
Copy link
Copy Markdown
Member

I updated the PR to rely upon code repository names to be as explicit as possible during this period where we do not necessarily have a clear and consistently used project name to described the mobile variant of the block editor and differences between the two repositories.

@dcalhoun dcalhoun requested a review from fluiddot January 10, 2024 14:27
@dcalhoun dcalhoun enabled auto-merge January 10, 2024 14:27
Copy link
Copy Markdown
Contributor

@fluiddot fluiddot left a comment

Choose a reason for hiding this comment

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

LGTM 🎊 ! Thanks @dcalhoun for updating the documentation 🙇 !

I noticed that some of the CI checks are failing. The error is the same we encountered yesterday related to Buildkite not being able to checkout the Gutenberg repository (#6520 (comment)). This is already fixed in trunk (#6525), so I'd recommend updating the branch with the latest changes.

@dcalhoun dcalhoun merged commit 62cbf88 into trunk Jan 10, 2024
@dcalhoun dcalhoun deleted the docs/clarify-e2e-test-repo-names branch January 10, 2024 15:43
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.

3 participants