Skip to content
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

Improve config_fast_builds.toml #11529

Merged
merged 2 commits into from
Jan 25, 2024

Conversation

doonv
Copy link
Contributor

@doonv doonv commented Jan 25, 2024

Objective

Theres no explaination for what the compiler flags config_fast_builds.toml do

Solution

Explain what the flags in config_fast_builds.toml do.

Copy link
Member

@BD103 BD103 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! Perhaps it would also be good to link to the Optimize your compilation times section in the Bevy Book.

@alice-i-cecile alice-i-cecile added C-Docs An addition or correction to our documentation A-Meta About the project itself labels Jan 25, 2024
Copy link
Member

@alice-i-cecile alice-i-cecile left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great :) Let's avoid backlinking for now: I'm worried about that going stale.

@alice-i-cecile alice-i-cecile added this pull request to the merge queue Jan 25, 2024
@alice-i-cecile alice-i-cecile added the S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it label Jan 25, 2024
Merged via the queue into bevyengine:main with commit a93cefe Jan 25, 2024
25 checks passed
tjamaan pushed a commit to tjamaan/bevy that referenced this pull request Feb 6, 2024
# Objective

Theres no explaination for what the compiler flags
`config_fast_builds.toml` do

## Solution

Explain what the flags in `config_fast_builds.toml` do.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Meta About the project itself C-Docs An addition or correction to our documentation S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants