bevyengine / bevyengine/bevy

Extend AppExit::Error(NonZero<u8>) to AppExit::Error(NonZero<u8>, Option<String>))

Open
#24,741 6 comments 0 reactions 0 assignees View on GitHub
A-App C-Feature D-Straightforward S-Needs-Review
Dominant language
Rust
Stars
48.2k
Forks
4.8k
Avg merge
3d 22h
Merged PRs (30d)
161

Description

## What problem does this solve or what need does it fill?

An `u8` often cannot describe the error well.

It will be helpfull if allowing to insert an `Option' for error message.

Especially for who need to call `update()` and handle error manually.

## What solution would you like?

Extend `AppExit::Error(NonZero)` to `AppExit::Error(NonZero, Option))`

## What alternative(s) have you considered?

Add a new `AppExit::Error(String)`.

Contributor guide

Open the contributing guide

Research direction

Start by searching for AppExit::Error and the update() call sites to understand how the existing variant is constructed and handled. Review the issue discussion before choosing between the requested optional message and the proposed String variant; done means the selected API supports descriptive errors without breaking or overlooking existing handling.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
game-dev
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
52/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.