Skip to content

Removing precompile #93

Merged
csvance merged 1 commit intomainfrom
remove-precompile
Jan 16, 2026
Merged

Removing precompile #93
csvance merged 1 commit intomainfrom
remove-precompile

Conversation

@csvance
Copy link
Collaborator

@csvance csvance commented Jan 16, 2026

Precompile depends on a gRPC server, and that means on either additional software or network dependencies.

Using a public server for precompile does not work on air gapped or restricted networks (see #81). I'm also not willing to increase the amount of dependencies just for precompile, perhaps that can eventually be done with an extension to take advantage of what is already installed.

Then there is also the matter of respecting user privacy. I meant to provide the precompile server endpoint as a public service, but I think many users may not be comfortable with this since it makes an outbound network connection.

For these reasons, I am removing precompile from the package. We may add it back as an extension in the future.

@codecov-commenter
Copy link

codecov-commenter commented Jan 16, 2026

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 84.02%. Comparing base (8176e94) to head (7f2383a).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #93      +/-   ##
==========================================
+ Coverage   83.48%   84.02%   +0.53%     
==========================================
  Files           7        7              
  Lines         557      532      -25     
==========================================
- Hits          465      447      -18     
+ Misses         92       85       -7     

☔ 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.

@csvance csvance merged commit bde341f into main Jan 16, 2026
13 checks passed
@csvance csvance deleted the remove-precompile branch January 16, 2026 20:11
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