Conversation
Signed-off-by: Christoph Schulze <christoph.schulze@polygon.io>
- Install uv on the fork fallback path so python-lint/test jobs that call uvx/uv run work outside the prebuilt images. - Replace is_some()+expect() in Reversed::validate with a let-else that bails, satisfying the workspace-wide clippy::expect_used = deny lint. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Signed-off-by: Christoph Schulze <christoph.schulze@polygon.io>
- Install flatc via setup-flatc in the generated-files job on the fork, matching how uv/cargo-hack are handled; upstream bakes it into the AMI. - Replace .clone() on Arc with Arc::clone(&..) in vortex-layout segment cache and scan task test to satisfy clippy::clone_on_ref_ptr. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Signed-off-by: Christoph Schulze <christoph.schulze@polygon.io>
- Fix rustdoc::private_intra_doc_links in reversed/mod.rs (drop link to private execute module) and rustdoc::redundant_explicit_links in erased.rs (use the shortcut intra-doc link form). - Gate the benchmark tuning + benchmark run steps in Python (test) to upstream; scripts/setup-benchmark.sh (aa-teardown etc.) requires the self-hosted prebuilt runner and aborts on GitHub-hosted ubuntu-latest. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Signed-off-by: Christoph Schulze <christoph.schulze@polygon.io>
cargo +nightly fmt (NIGHTLY_TOOLCHAIN) normalizes a blank line inside the doc-comment diagram in patched/mod.rs; commit the formatting so 'Rust (lint) - Format' (cargo fmt --all --check) passes. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Signed-off-by: Christoph Schulze <christoph.schulze@polygon.io>
docs/conf.py requires doxygen on PATH to build the C++ API docs via breathe; it is preinstalled on the prebuilt images but missing on GitHub-hosted runners, so 'make doctest'/'make html' aborted. Install it via ssciwr/doxygen-install on the fork before the docs steps. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Signed-off-by: Christoph Schulze <christoph.schulze@polygon.io>
- Bump crossbeam-epoch 0.9.18 -> 0.9.20 (RUSTSEC-2026-0204, patched >= 0.9.20), matching upstream develop. - Bump cxx 1.0.194 -> 1.0.195 (RUSTSEC-2026-0202, patched >= 1.0.195). - Ignore RUSTSEC-2026-0194/0195 (quick-xml) in deny.toml; no patched release exists and it is a transitive dependency, matching upstream. cargo deny check advisories now passes. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Signed-off-by: Christoph Schulze <christoph.schulze@polygon.io>
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
Summary
Closes: #000
Testing