stride3d / stride3d/stride

[Physics] Better naming scheme for PhysicsComponent.ColliderShape-s

Open
#1,473 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

enhancement
Dominant language
C#
Stars
7.8k
Forks
1.2k
Avg merge
2d 17h
Merged PRs (30d)
49

Description

Release Type: Latest as of writing

PhysicsComponent has both ColliderShape and ColliderShapes, how each type differs is not as obvious as it should be.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start with sources/engine/Stride.Physics/Engine/PhysicsComponent.cs around the ColliderShape and ColliderShapes declarations. Read how both names are used to determine the distinction, then define and apply a clearer naming scheme and verify that the two concepts remain unambiguous throughout the component API.

Written by the indexing model from the issue text.

Assessment

Tech stack
csharp
Domain
game-dev
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.