-
Notifications
You must be signed in to change notification settings - Fork 188
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(torii): sync from master torii #2851
base: main
Are you sure you want to change the base?
Conversation
WalkthroughOhayo, sensei! This pull request introduces configuration enhancements for replica options and a new import for Changes
Possibly related PRs
Suggested reviewers
Sensei, the changes look like they're setting up some exciting groundwork for more flexible replica configurations! 🌍🔧 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
Documentation and Community
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
🧹 Nitpick comments (1)
crates/torii/cli/src/options.rs (1)
325-350
: Ohayo sensei! The newReplicaOptions
struct is a neat addition.
- Great approach to using clap to parse CLI arguments.
- The default implementation is consistent with the newly introduced constants.
Overall, this extends the configuration flexibility without clutter. Just ensure code coverage with tests, especially for scenarios where users override these defaults.
📜 Review details
Configuration used: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (2)
bin/torii/src/main.rs
(1 hunks)crates/torii/cli/src/options.rs
(3 hunks)
🔇 Additional comments (4)
bin/torii/src/main.rs (1)
42-42
: Ohayo sensei! This import looks good.
The addition of WorldClient
import sets the stage for gRPC interactions with the world. No issues spotted here.
crates/torii/cli/src/options.rs (3)
5-5
: Ohayo sensei! This import broadens parse utilities.
Importing parse_url
appears consistent with new usage in ReplicaOptions
. Good to keep code DRY by reusing an existing parse function.
10-10
: Ohayo sensei! The Url
import is necessary to validate external endpoints.
This ensures robust URL parsing for the new config struct. No concerns spotted.
25-26
: Ohayo sensei! Defining new default constants for replica behavior.
The default true
values are likely safe for most scenarios, ensuring entity and event replication by default. Just confirm if this aligns with user expectations.
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #2851 +/- ##
==========================================
- Coverage 56.16% 56.16% -0.01%
==========================================
Files 440 440
Lines 56635 56647 +12
==========================================
+ Hits 31810 31815 +5
- Misses 24825 24832 +7 ☔ View full report in Codecov by Sentry. |
Summary by CodeRabbit
New Features
Chores
WorldClient
fromtorii_grpc