bevyengine / bevyengine/bevy

Consider renaming `EnvironmentMapLight` to `EnvironmentLight`

Open
#20,444 7 comments 0 reactions 0 assignees View on GitHub
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

Open the contributing 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.