Skip to content

Conversation

TestingPlant
Copy link
Collaborator

No description provided.

@github-actions github-actions bot added the docs label Sep 27, 2025
Copy link

Benchmark Results for general

ray_intersection/aabb_size_0.1                     [  19.2 ns ...  19.2 ns ]      -0.11%
ray_intersection/aabb_size_1                       [  19.2 ns ...  19.2 ns ]      -0.22%
ray_intersection/aabb_size_10                      [  19.2 ns ...  19.2 ns ]      -0.00%
ray_intersection/ray_distance_1                    [   3.8 ns ...   3.8 ns ]      -0.13%
ray_intersection/ray_distance_5                    [   3.8 ns ...   3.8 ns ]      -0.16%
ray_intersection/ray_distance_20                   [   3.8 ns ...   3.8 ns ]      +0.04%
overlap/no_overlap                                 [  29.1 ns ...  29.1 ns ]      -0.05%
overlap/partial_overlap                            [  28.4 ns ...  28.5 ns ]      +0.05%
overlap/full_containment                           [  21.7 ns ...  21.6 ns ]      -0.04%
point_containment/inside                           [   4.9 ns ...   4.9 ns ]      -0.15%
point_containment/outside                          [   4.4 ns ...   4.4 ns ]      -0.12%
point_containment/boundary                         [   5.0 ns ...   5.0 ns ]      -0.14%

Comparing to 1f28877

Copy link

codecov bot commented Sep 27, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 23.22%. Comparing base (1f28877) to head (1f4644b).

@@           Coverage Diff           @@
##             main     #947   +/-   ##
=======================================
  Coverage   23.22%   23.22%           
=======================================
  Files         157      157           
  Lines       15494    15494           
  Branches      486      486           
=======================================
  Hits         3599     3599           
  Misses      11794    11794           
  Partials      101      101           
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Comment on lines +188 to +202
```bash
openssl req -new -nodes -newkey rsa:4096 -keyout root_ca.pem -x509 -out root_ca.crt -days 365 -subj /
openssl req -nodes -newkey rsa:4096 -keyout game_private_key.pem -out game.csr -subj /
openssl x509 -req -in game.csr -CA root_ca.crt -CAkey root_ca.pem -CAcreateserial -out game.crt -days 365 -sha256 -extfile <(printf "subjectAltName=DNS:localhost,IP:127.0.0.1")
rm game.csr
openssl req -nodes -newkey rsa:4096 -keyout proxy_private_key.pem -out proxy.csr -subj /
openssl x509 -req -in proxy.csr -CA root_ca.crt -CAkey root_ca.pem -CAcreateserial -out proxy.crt -days 365 -sha256 -extfile <(printf "subjectAltName=DNS:localhost,IP:127.0.0.1")
rm proxy.csr
```

Now run the game server. Note that the game server automatically starts the proxy.

```bash
cargo run --release --bin bedwars -- --ip 127.0.0.1 --root-ca-cert root_ca.crt --cert game.crt --private-key game_private_key.pem
```
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

since these are for a dev environment, maybe it would be good to include some of these long commands in the justfile?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd like to avoid relying on external programs like just that people usually don't have installed so that setting up Hyperion is simpler

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not saying remove these instructions necessarily, merely providing the convenience of running a simple command.

My full opinion: installing just is not a hurdle, its a cargo install just and then wait. providing an easy way to have consistent dev environments is way more valuable than the time spent debugging differences in environments, multiplied by however many people start contributing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants