Skip to content

revert: restore archive-completeness check in the MCP manifest test#71707

Merged
gewenyu99 merged 1 commit into
masterfrom
revert-71694-mcp-manifest-test
Jul 16, 2026
Merged

revert: restore archive-completeness check in the MCP manifest test#71707
gewenyu99 merged 1 commit into
masterfrom
revert-71694-mcp-manifest-test

Conversation

@gewenyu99

@gewenyu99 gewenyu99 commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Reverts test updated in #248. Fixed at source (context mill)

@github-actions

Copy link
Copy Markdown
Contributor

Hey @gewenyu99! 👋

It looks like your git author email on this PR isn't your @posthog.com address (gewenyu99@gmail.com). Since you're on the PostHog team, it's worth pointing your local git author email at your @posthog.com address. Why it matters:

  • Consistent work identity in git history — internal tooling that attributes commits to team members keys off your @posthog.com address.
  • Keeps team contributions easy to tell apart from external community ones when scanning history.

You can fix it for this repo with:

git config user.email "you@posthog.com"

Or set it globally with git config --global user.email "you@posthog.com". No need to redo this PR — just a nudge for next time. 🙂

@gewenyu99 gewenyu99 changed the title Revert "fix(mcp): handle resources absent from aggregate archive" revert: restore archive-completeness check in the MCP manifest test Jul 16, 2026
@gewenyu99
gewenyu99 marked this pull request as ready for review July 16, 2026 19:08
@gewenyu99
gewenyu99 requested a review from pauldambra July 16, 2026 19:13
@gewenyu99
gewenyu99 enabled auto-merge (squash) July 16, 2026 19:18
@gewenyu99
gewenyu99 merged commit d6514d4 into master Jul 16, 2026
280 of 283 checks passed
@gewenyu99
gewenyu99 deleted the revert-71694-mcp-manifest-test branch July 16, 2026 19:18
@deployment-status-posthog

deployment-status-posthog Bot commented Jul 16, 2026

Copy link
Copy Markdown

Deploy status

Environment Status Deployed At Workflow
dev ✅ Deployed 2026-07-16 19:51 UTC Run
prod-us ✅ Deployed 2026-07-16 20:12 UTC Run
prod-eu ✅ Deployed 2026-07-16 20:12 UTC Run

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.

2 participants