Consider renaming `EnvironmentMapLight` to `EnvironmentLight`
Open
A-Rendering
C-Usability
S-Blocked
X-Contentious
- Dominant language
- Rust
- Stars
- 48.2k
- Forks
- 4.8k
- Avg merge
- 3d 16h
- Merged PRs (30d)
- 171
Description
`EnvironmentMapLight` is wordy and post-deprecation of `AmbientLight` in #18207 will be more commonly used. We should prefer simpler, shorter names unless the longer name provides sufficiently important descriptive information.
Contributor guide
Research direction
Start by searching the repository for references to `EnvironmentMapLight` and `AmbientLight`, then read the discussion linked as #18207. Determine whether the shorter name preserves the needed meaning and identify the full set of affected references; done means the naming decision and its consequences are documented clearly enough for implementation.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- rust
- Domain
- computer-graphics, game-dev
- Issue type
- Refactor
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100