-
-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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
[Merged by Bors] - Document setting "CARGO_MANIFEST_DIR" for asset root #1950
[Merged by Bors] - Document setting "CARGO_MANIFEST_DIR" for asset root #1950
Conversation
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.
IMO this explanation also belongs (in a terse form) on the AssetServer::load method, to make sure it's discoverable on docs.rs.
would be changed by #1801 |
That would only be a minor change to adapt, as the consensus is to use the working directory as a Fallback when the exe directory doesn't work. |
Co-authored-by: bjorn3 <[email protected]>
bors r+ |
This was nowhere documented inside Bevy. Should I also mention the use case of debugging a project? Closes #810 Co-authored-by: MinerSebas <[email protected]>
Pull request successfully merged into main. Build succeeded: |
This was nowhere documented inside Bevy. Should I also mention the use case of debugging a project? Closes bevyengine#810 Co-authored-by: MinerSebas <[email protected]>
This was nowhere documented inside Bevy.
Should I also mention the use case of debugging a project?
Closes #810