Skip to content

Remove the new rust-based website from the workspace#8246

Closed
AdamGS wants to merge 1 commit into
developfrom
adamg/remove-rust-website-workspace
Closed

Remove the new rust-based website from the workspace#8246
AdamGS wants to merge 1 commit into
developfrom
adamg/remove-rust-website-workspace

Conversation

@AdamGS

@AdamGS AdamGS commented Jun 4, 2026

Copy link
Copy Markdown
Contributor

Summary

This simplifies the dependency conflicts/issues, and also a step towards moving the website into its own repo.

Signed-off-by: Adam Gutglick <adam@spiraldb.com>
@AdamGS

AdamGS commented Jun 4, 2026

Copy link
Copy Markdown
Contributor Author

@0ax1 @myrrc is this what you guys had in mind?

@AdamGS AdamGS added the changelog/chore A trivial change label Jun 4, 2026
@codspeed-hq

codspeed-hq Bot commented Jun 4, 2026

Copy link
Copy Markdown

Merging this PR will improve performance by 30.19%

⚠️ Unknown Walltime execution environment detected

Using the Walltime instrument on standard Hosted Runners will lead to inconsistent data.

For the most accurate results, we recommend using CodSpeed Macro Runners: bare-metal machines fine-tuned for performance measurement consistency.

⚡ 3 improved benchmarks
✅ 1504 untouched benchmarks

Performance Changes

Mode Benchmark BASE HEAD Efficiency
Simulation chunked_bool_canonical_into[(1000, 10)] 45.3 µs 30.3 µs +49.41%
Simulation chunked_varbinview_canonical_into[(1000, 10)] 197.1 µs 160.8 µs +22.53%
Simulation chunked_varbinview_into_canonical[(1000, 10)] 211.7 µs 175.6 µs +20.54%

Tip

Curious why this is faster? Comment @codspeedbot explain why this is faster on this PR, or directly use the CodSpeed MCP with your agent.


Comparing adamg/remove-rust-website-workspace (6663e70) with develop (4e6e9ed)

Open in CodSpeed

@connortsui20

Copy link
Copy Markdown
Member

Let's just kill the entire thing, though that requires removing a bunch of other stuff in CI.

The new version I'm vibe coding in the background can live in a different place if we want.

Please let me know before we merge so I can update my ec2 server

@AdamGS

AdamGS commented Jun 8, 2026

Copy link
Copy Markdown
Contributor Author

are we replacing vibe-web-server-rs 1.0 with vibe-web-server-rs 2.0? If there's more work to do that, do you mind taking the work to remove the current version from the repo?

@AdamGS AdamGS closed this Jun 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog/chore A trivial change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants