Skip to content

Prevent merging PRs that sync from upstream #103296

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: trunk
Choose a base branch
from

Conversation

oandregal
Copy link
Contributor

@oandregal oandregal commented May 9, 2025

DOTDEV-130
Related #103247

Proposed Changes

This PR adds a TeamCity check so that if there are changes to the packages/dataviews/CHANGELOG.md the job fails.

Why are these changes being made?

PRs that change packages/dataviews/CHANGELOG.md can only be PRs that sync changes from upstream Gutenberg. We don't want to merge these PRs via the GitHub UI merge button, but rather via the CLI command described at packages/dataviews/SYNC.md.

Testing Instructions

This was referenced May 9, 2025
@oandregal oandregal requested a review from mcsf May 9, 2025 15:43
@matticbot matticbot added the [Status] Needs Review The PR is ready for review. This also triggers e2e canary tests and wp-desktop tests automatically. label May 9, 2025
@oandregal oandregal requested review from BogdanUngureanu and a team May 9, 2025 15:43
@oandregal oandregal self-assigned this May 9, 2025
@oandregal oandregal mentioned this pull request May 9, 2025
8 tasks
@matticbot
Copy link
Contributor

This PR does not affect the size of JS and CSS bundles shipped to the user's browser.

Generated by performance advisor bot at iscalypsofastyet.com.

@matticbot
Copy link
Contributor

This PR modifies the release build for the following Calypso Apps:

For info about this notification, see here: PCYsg-OT6-p2

  • notifications
  • wpcom-block-editor

To test WordPress.com changes, run install-plugin.sh $pluginSlug add/tc-check-to-prevent-pr on your sandbox.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Status] Needs Review The PR is ready for review. This also triggers e2e canary tests and wp-desktop tests automatically.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants