Skip to content

Conversation

@ericallam
Copy link
Member

No description provided.

@changeset-bot
Copy link

changeset-bot bot commented Feb 12, 2026

⚠️ No Changeset found

Latest commit: f4d1fc7

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 12, 2026

Walkthrough

The .github/VOUCHED.td file was modified to reorder the contributor list. The contributor isshaddad was removed from the Org members section and repositioned above the "# Outside contributors" heading. Additionally, gautamsi was added to the Outside contributors section. The overall content structure and contributors remain the same, with only positional adjustments applied.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 2
❌ Failed checks (2 warnings)
Check name Status Explanation Resolution
Description check ⚠️ Warning The PR description is completely empty, missing all required template sections including checklist, testing details, and changelog. Add a complete PR description following the repository template with checklist items, testing section, and changelog entry.
Merge Conflict Detection ⚠️ Warning ❌ Merge conflicts detected (66 files):

⚔️ .github/VOUCHED.td (content)
⚔️ .vscode/settings.json (content)
⚔️ apps/webapp/app/components/AlphaBadge.tsx (content)
⚔️ apps/webapp/app/components/code/AIQueryInput.tsx (content)
⚔️ apps/webapp/app/components/code/ChartConfigPanel.tsx (content)
⚔️ apps/webapp/app/components/code/QueryResultsChart.tsx (content)
⚔️ apps/webapp/app/components/code/TSQLEditor.tsx (content)
⚔️ apps/webapp/app/components/code/TSQLResultsTable.tsx (content)
⚔️ apps/webapp/app/components/code/tsql/tsqlCompletion.ts (content)
⚔️ apps/webapp/app/components/layout/AppLayout.tsx (content)
⚔️ apps/webapp/app/components/logs/LogsTaskFilter.tsx (content)
⚔️ apps/webapp/app/components/navigation/HelpAndFeedbackPopover.tsx (content)
⚔️ apps/webapp/app/components/navigation/OrganizationSettingsSideMenu.tsx (content)
⚔️ apps/webapp/app/components/navigation/SideMenu.tsx (content)
⚔️ apps/webapp/app/components/navigation/SideMenuItem.tsx (content)
⚔️ apps/webapp/app/components/navigation/SideMenuSection.tsx (content)
⚔️ apps/webapp/app/components/primitives/AppliedFilter.tsx (content)
⚔️ apps/webapp/app/components/primitives/ClientTabs.tsx (content)
⚔️ apps/webapp/app/components/primitives/FormButtons.tsx (content)
⚔️ apps/webapp/app/components/primitives/LoadingBarDivider.tsx (content)
⚔️ apps/webapp/app/components/primitives/Popover.tsx (content)
⚔️ apps/webapp/app/components/primitives/Resizable.tsx (content)
⚔️ apps/webapp/app/components/primitives/Tooltip.tsx (content)
⚔️ apps/webapp/app/components/primitives/charts/Card.tsx (content)
⚔️ apps/webapp/app/components/primitives/charts/ChartBar.tsx (content)
⚔️ apps/webapp/app/components/primitives/charts/ChartLegendCompound.tsx (content)
⚔️ apps/webapp/app/components/primitives/charts/ChartLine.tsx (content)
⚔️ apps/webapp/app/components/runs/v3/SharedFilters.tsx (content)
⚔️ apps/webapp/app/components/runs/v3/TaskRunStatus.tsx (content)
⚔️ apps/webapp/app/env.server.ts (content)
⚔️ apps/webapp/app/hooks/useOrganizations.ts (content)
⚔️ apps/webapp/app/models/runtimeEnvironment.server.ts (content)
⚔️ apps/webapp/app/presenters/v3/LimitsPresenter.server.ts (content)
⚔️ apps/webapp/app/routes/_app.orgs.$organizationSlug.projects.$projectParam.env.$envParam.limits/route.tsx (content)
⚔️ apps/webapp/app/routes/_app.orgs.$organizationSlug.projects.$projectParam.env.$envParam.query/AITabContent.tsx (content)
⚔️ apps/webapp/app/routes/_app.orgs.$organizationSlug.projects.$projectParam.env.$envParam.query/ExamplesContent.tsx (content)
⚔️ apps/webapp/app/routes/_app.orgs.$organizationSlug.projects.$projectParam.env.$envParam.query/QueryHistoryPopover.tsx (content)
⚔️ apps/webapp/app/routes/_app.orgs.$organizationSlug.projects.$projectParam.env.$envParam.query/TRQLGuideContent.tsx (content)
⚔️ apps/webapp/app/routes/_app.orgs.$organizationSlug.projects.$projectParam.env.$envParam.query/route.tsx (content)
⚔️ apps/webapp/app/routes/_app.orgs.$organizationSlug/route.tsx (content)
⚔️ apps/webapp/app/routes/resources.incidents.tsx (content)
⚔️ apps/webapp/app/routes/resources.orgs.$organizationSlug.select-plan.tsx (content)
⚔️ apps/webapp/app/routes/resources.preferences.sidemenu.tsx (content)
⚔️ apps/webapp/app/routes/storybook.charts/route.tsx (content)
⚔️ apps/webapp/app/services/betterstack/betterstack.server.ts (content)
⚔️ apps/webapp/app/services/clickhouseInstance.server.ts (content)
⚔️ apps/webapp/app/services/dashboardPreferences.server.ts (content)
⚔️ apps/webapp/app/services/queryService.server.ts (content)
⚔️ apps/webapp/app/tailwind.css (content)
⚔️ apps/webapp/app/utils/pathBuilder.ts (content)
⚔️ apps/webapp/app/v3/querySchemas.ts (content)
⚔️ apps/webapp/app/v3/services/aiQueryService.server.ts (content)
⚔️ apps/webapp/package.json (content)
⚔️ apps/webapp/tailwind.config.js (content)
⚔️ internal-packages/clickhouse/src/client/tsql.ts (content)
⚔️ internal-packages/database/prisma/schema.prisma (content)
⚔️ internal-packages/tsql/src/index.ts (content)
⚔️ internal-packages/tsql/src/query/printer.test.ts (content)
⚔️ internal-packages/tsql/src/query/printer.ts (content)
⚔️ internal-packages/tsql/src/query/printer_context.ts (content)
⚔️ internal-packages/tsql/src/query/schema.ts (content)
⚔️ internal-packages/tsql/src/query/validator.ts (content)
⚔️ packages/cli-v3/src/mcp/config.ts (content)
⚔️ packages/cli-v3/src/mcp/schemas.ts (content)
⚔️ packages/cli-v3/src/mcp/tools/runs.ts (content)
⚔️ pnpm-lock.yaml (content)

These conflicts must be resolved before merging into main.
Resolve conflicts locally and push changes to this branch.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and specifically describes the main change: vouching for a contributor (gautamsi) in the VOUCHED.td file.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch ea-branch-118
⚔️ Resolve merge conflicts (beta)
  • Auto-commit resolved conflicts to branch ea-branch-118
  • Create stacked PR with resolved conflicts
  • Post resolved changes as copyable diffs in a comment

No actionable comments were generated in the recent review. 🎉

📜 Recent review details

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between bc0d1ff and f4d1fc7.

📒 Files selected for processing (1)
  • .github/VOUCHED.td
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (28)
  • GitHub Check: units / webapp / 🧪 Unit Tests: Webapp (5, 8)
  • GitHub Check: units / internal / 🧪 Unit Tests: Internal (1, 8)
  • GitHub Check: units / webapp / 🧪 Unit Tests: Webapp (6, 8)
  • GitHub Check: units / webapp / 🧪 Unit Tests: Webapp (8, 8)
  • GitHub Check: units / webapp / 🧪 Unit Tests: Webapp (2, 8)
  • GitHub Check: units / webapp / 🧪 Unit Tests: Webapp (7, 8)
  • GitHub Check: units / internal / 🧪 Unit Tests: Internal (2, 8)
  • GitHub Check: units / internal / 🧪 Unit Tests: Internal (6, 8)
  • GitHub Check: units / webapp / 🧪 Unit Tests: Webapp (4, 8)
  • GitHub Check: units / webapp / 🧪 Unit Tests: Webapp (3, 8)
  • GitHub Check: units / internal / 🧪 Unit Tests: Internal (8, 8)
  • GitHub Check: units / webapp / 🧪 Unit Tests: Webapp (1, 8)
  • GitHub Check: units / internal / 🧪 Unit Tests: Internal (7, 8)
  • GitHub Check: units / internal / 🧪 Unit Tests: Internal (5, 8)
  • GitHub Check: units / internal / 🧪 Unit Tests: Internal (3, 8)
  • GitHub Check: units / internal / 🧪 Unit Tests: Internal (4, 8)
  • GitHub Check: units / packages / 🧪 Unit Tests: Packages (1, 1)
  • GitHub Check: e2e / 🧪 CLI v3 tests (ubuntu-latest - pnpm)
  • GitHub Check: e2e / 🧪 CLI v3 tests (windows-latest - pnpm)
  • GitHub Check: e2e / 🧪 CLI v3 tests (ubuntu-latest - npm)
  • GitHub Check: sdk-compat / Cloudflare Workers
  • GitHub Check: e2e / 🧪 CLI v3 tests (windows-latest - npm)
  • GitHub Check: sdk-compat / Node.js 22.12 (ubuntu-latest)
  • GitHub Check: sdk-compat / Bun Runtime
  • GitHub Check: typecheck / typecheck
  • GitHub Check: sdk-compat / Deno Runtime
  • GitHub Check: sdk-compat / Node.js 20.20 (ubuntu-latest)
  • GitHub Check: Analyze (javascript-typescript)
🔇 Additional comments (1)
.github/VOUCHED.td (1)

13-15: LGTM!

The addition of gautamsi under Outside contributors aligns with the PR objective. The minor reordering of isshaddad within the Org members section has no functional impact.

✏️ Tip: You can disable this entire section by setting review_details to false in your review settings.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Open in Devin Review

@nicktrn nicktrn merged commit ba320a4 into main Feb 12, 2026
36 checks passed
@nicktrn nicktrn deleted the ea-branch-118 branch February 12, 2026 22:27
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