Skip to content

SRE-142: Remove DeepSource - #8041

Merged
indietyp merged 2 commits into
mainfrom
bm/sre-142-remove-deepsource-from-hash-repo
Nov 18, 2025
Merged

SRE-142: Remove DeepSource#8041
indietyp merged 2 commits into
mainfrom
bm/sre-142-remove-deepsource-from-hash-repo

Conversation

@indietyp

Copy link
Copy Markdown
Member

🌟 What is the purpose of this PR?

Remove DeepSource integration and make sccache installation optional in CI workflows

🔍 What does this change?

  • Removes the .deepsource.toml configuration file
  • Makes sccache installation optional in the install-tools GitHub Action
  • Adds a new sccache input parameter to the install-tools action (defaults to "true")
  • Updates the bench workflow to avoid installing sccache twice
  • Removes DeepSource test coverage upload from the test workflow
  • Removes Vault secret retrieval for DeepSource from the test workflow

Pre-Merge Checklist 🚀

🚢 Has this modified a publishable library?

This PR:

  • does not modify any publishable blocks or libraries, or modifications do not need publishing

📜 Does this require a change to the docs?

The changes in this PR:

  • are internal and do not require a docs change

🕸️ Does this require a change to the Turbo Graph?

The changes in this PR:

  • do not affect the execution graph

🛡 What tests cover this?

  • CI workflows will validate these changes automatically

❓ How to test this?

  1. Checkout the branch
  2. Verify CI workflows run successfully without DeepSource integration
  3. Confirm that sccache is properly installed when needed

@vercel

vercel Bot commented Nov 18, 2025

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Updated (UTC)
petrinaut Error Error Nov 18, 2025 8:46am

@github-actions github-actions Bot added the area/infra Relates to version control, CI, CD or IaC (area) label Nov 18, 2025
@vercel
vercel Bot temporarily deployed to Preview – petrinaut November 18, 2025 08:46 Inactive

indietyp commented Nov 18, 2025

Copy link
Copy Markdown
Member Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@deepsource-io

deepsource-io Bot commented Nov 18, 2025

Copy link
Copy Markdown

Here's the code health analysis summary for commits d549b46..b8178a2. View details on DeepSource ↗.

Analysis Summary

AnalyzerStatusSummaryLink
DeepSource JavaScript LogoJavaScript✅ SuccessView Check ↗
DeepSource Secrets LogoSecrets✅ SuccessView Check ↗
DeepSource SQL LogoSQL✅ SuccessView Check ↗
DeepSource Test coverage LogoTest coverage⚠️ Artifact not reportedTimed out: Artifact was never reportedView Check ↗
DeepSource Rust LogoRust✅ SuccessView Check ↗
DeepSource Shell LogoShell✅ SuccessView Check ↗
DeepSource Docker LogoDocker✅ SuccessView Check ↗

💡 If you’re a repository administrator, you can configure the quality gates from the settings.

@vercel
vercel Bot temporarily deployed to Preview – hashdotdesign November 18, 2025 08:49 Inactive
@codecov

codecov Bot commented Nov 18, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 55.85%. Comparing base (081d3d4) to head (b8178a2).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #8041   +/-   ##
=======================================
  Coverage   55.85%   55.85%           
=======================================
  Files        1142     1142           
  Lines      105678   105678           
  Branches     4826     4826           
=======================================
+ Hits        59026    59027    +1     
+ Misses      45957    45956    -1     
  Partials      695      695           
Flag Coverage Δ
apps.hash-ai-worker-ts 1.32% <ø> (ø)
apps.hash-api 0.00% <ø> (ø)
blockprotocol.type-system 40.84% <ø> (ø)
local.claude-hooks 0.00% <ø> (ø)
local.harpc-client 51.24% <ø> (ø)
local.hash-graph-sdk 10.88% <ø> (ø)
local.hash-isomorphic-utils 0.00% <ø> (ø)
rust.antsi 0.00% <ø> (ø)
rust.error-stack 90.85% <ø> (ø)
rust.harpc-codec 84.22% <ø> (ø)
rust.harpc-net 96.13% <ø> (+0.01%) ⬆️
rust.harpc-tower 66.80% <ø> (ø)
rust.harpc-types 0.00% <ø> (ø)
rust.harpc-wire-protocol 92.23% <ø> (ø)
rust.hash-codec 71.25% <ø> (ø)
rust.hash-graph-api 2.89% <ø> (ø)
rust.hash-graph-authorization 62.50% <ø> (ø)
rust.hash-graph-postgres-store 25.62% <ø> (ø)
rust.hash-graph-store 30.70% <ø> (ø)
rust.hash-graph-temporal-versioning 47.68% <ø> (ø)
rust.hash-graph-types 0.00% <ø> (ø)
rust.hash-graph-validation 83.29% <ø> (ø)
rust.hashql-ast 87.17% <ø> (ø)
rust.hashql-compiletest 48.96% <ø> (ø)
rust.hashql-core 81.01% <ø> (ø)
rust.hashql-diagnostics 72.28% <ø> (ø)
rust.hashql-eval 68.56% <ø> (ø)
rust.hashql-hir 88.92% <ø> (ø)
rust.hashql-mir 54.38% <ø> (ø)
rust.hashql-syntax-jexpr 93.94% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@indietyp
indietyp added this pull request to the merge queue Nov 18, 2025
@graphite-app
graphite-app Bot requested a review from a team November 18, 2025 09:08
Merged via the queue into main with commit 6f35d2f Nov 18, 2025
170 of 173 checks passed
@indietyp
indietyp deleted the bm/sre-142-remove-deepsource-from-hash-repo branch November 18, 2025 09:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/infra Relates to version control, CI, CD or IaC (area)

Development

Successfully merging this pull request may close these issues.

3 participants