Skip to content

chore: remove Vercel docs deploy CI jobs#360

Merged
tonyxiao merged 2 commits intomainfrom
copilot/remove-old-vercel-ci-docs
May 5, 2026
Merged

chore: remove Vercel docs deploy CI jobs#360
tonyxiao merged 2 commits intomainfrom
copilot/remove-old-vercel-ci-docs

Conversation

Copy link
Copy Markdown

Copilot AI commented May 5, 2026

Summary

The Vercel project that hosted the docs site no longer exists. Removes the two CI jobs that depended on it:

  • docs — built and deployed docs/slides/visualizer to Vercel on every run (VERCEL_TOKEN, VERCEL_ORG_ID, VERCEL_PROJECT_ID)
  • e2e_cdn — ran CDN integration tests against the Vercel deployment URL (stripe-sync.dev/stripe-api-specs)

No other jobs had a needs dependency on either.

How to test (optional)

Related

Thanks for contributing ❤️

Original prompt

Create a pull request in stripe/sync-engine-fork that removes the CI/docs deploy configuration related to the old Vercel project, since that Vercel project no longer exists.

Context:

  • The user said: "that vercel project is no more i think. can u just create a new pr that removes this ci deploy docs thing?"
  • The repository was confirmed as stripe/sync-engine-fork.
  • Relevant reference PR: Add source and destination for custom object support #356
  • Use main as the base branch.

Task:

  • Identify the workflow/config/docs deployment pieces in this repository that publish or deploy docs via CI and rely on the old Vercel project.
  • Remove or disable that CI/docs deploy setup cleanly.
  • Update any related docs/comments/configuration as needed so the repo no longer references or expects that deploy path.
  • Keep the change narrowly scoped to removing the obsolete docs deploy/Vercel integration.

Deliverable:

  • Open a new pull request with the changes.

The following is the prior conversation context from the user's chat exploration (may be truncated):

User: that vercel project is no more i think. can u just create a new pr that removes this ci deploy docs thing?

Assistant: Yes — I can do that, but I need the target repository explicitly confirmed.

You linked this PR:

  • stripe/sync-engine-fork PR #356

If you want, I can create a new PR in:

  • stripe/sync-engine-fork

that removes the CI/docs deploy thing that references the old Vercel project.

Reply with something like:

  • Yes, create it in stripe/sync-engine-fork
  • optionally: base branch is main or another branch if not the default

Once you confirm the repo, I’ll open the PR.

User: yes create it in the repo

Assistant: I’ve started the PR creation process and we’re waiting for your response to the confirmation dialog to continue.

This pull request was created from Copilot chat.

Agent-Logs-Url: https://github.com/stripe/sync-engine-fork/sessions/cd9648bf-cb04-4410-915a-0c8b46e2bf0a

Co-authored-by: kdhillon-stripe <243457111+kdhillon-stripe@users.noreply.github.com>
@cla-assistant
Copy link
Copy Markdown

cla-assistant Bot commented May 5, 2026

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

1 similar comment
@cla-assistant
Copy link
Copy Markdown

cla-assistant Bot commented May 5, 2026

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

Copilot AI changed the title [WIP] Remove CI/docs deploy configuration for old Vercel project chore: remove Vercel docs deploy CI jobs May 5, 2026
Copilot AI requested a review from kdhillon-stripe May 5, 2026 07:45
@kdhillon-stripe kdhillon-stripe marked this pull request as ready for review May 5, 2026 07:52
Copilot AI review requested due to automatic review settings May 5, 2026 07:52
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Removes the obsolete GitHub Actions CI jobs that deployed the docs site to Vercel and then ran CDN E2E tests against that deployment, since the Vercel project no longer exists.

Changes:

  • Deleted the docs workflow job that built and deployed docs/ (including slides + visualizer) to Vercel.
  • Deleted the e2e_cdn workflow job that depended on the Vercel deployment URL to run CDN integration tests.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Copy Markdown
Collaborator

@tonyxiao tonyxiao left a comment

Choose a reason for hiding this comment

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

should bring this back some point. but sog for now

@tonyxiao tonyxiao merged commit 95eaaa9 into main May 5, 2026
21 of 22 checks passed
@tonyxiao tonyxiao deleted the copilot/remove-old-vercel-ci-docs branch May 5, 2026 08:15
tonyxiao pushed a commit that referenced this pull request May 7, 2026
Reverts back to 5024aac (change soft to hard delete).

Reverted commits:
- 921fe63 Add source and destination for custom object support (#356)
- 95eaaa9 chore: remove Vercel docs deploy CI jobs (#360)
- 822000f squash (#352)


Committed-By-Agent: claude

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants