Update libLLVM download URL and clean up configuration files #88
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
… script, and clean up configuration
…dd platform-specific setup instructions
There was a problem hiding this comment.
Pull request overview
This PR updates the libLLVM download URL to include a -nightly suffix in the filename and removes unused playground-server tool and configuration files. The changes improve consistency across the codebase by standardizing the LLVM library naming convention and cleaning up deprecated development infrastructure.
Changes:
- Updated libLLVM download URLs from
libLLVM.so.21.1-rust-1.94.ziptolibLLVM.so.21.1-rust-1.94.0-nightly.zipacross multiple files - Removed the unused
playground-servertool and its Cargo manifest - Updated LLVM environment variable from
LLVM_SYS_210_PREFIXtoLLVM_SYS_211_PREFIXin Windows documentation - Added comprehensive Linux and macOS setup guides with platform-specific instructions
- Added a cross-platform dependency check script
- Removed unused PATH configuration from
.cargo/config.toml - Bumped CI cache keys to invalidate stale binary caches
Reviewed changes
Copilot reviewed 13 out of 13 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| tools/playground-server/src/main.rs | Removed entire playground server implementation |
| tools/playground-server/Cargo.toml | Removed Cargo manifest for playground server |
| core/wasm-codegen/build.rs | Updated libLLVM download URL with -nightly suffix |
| book/installation_windows.md | Updated LLVM environment variable to version 211 |
| book/installation_macos.md | Added comprehensive macOS development setup guide |
| book/installation_linux.md | Added comprehensive Linux development setup guide |
| book/check_deps.sh | Added cross-platform dependency verification script |
| README.md | Updated libLLVM download URL and added links to platform guides |
| Cargo.toml | Removed playground-server workspace member |
| CHANGELOG.md | Documented all changes in this PR |
| .github/workflows/reusable-build.yml | Updated libLLVM URL and cache key |
| .github/workflows/coverage.yml | Updated libLLVM URL and cache key |
| .cargo/config.toml | Removed unused PATH environment configuration |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Closes #56