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.

Sign up for GitHub

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

Jump to bottom

make alien_cake_addict deterministic with a seeded random #12515

Merged
merged 2 commits into from
Mar 17, 2024
Merged

make alien_cake_addict deterministic with a seeded random #12515

merged 2 commits into from
Mar 17, 2024

Conversation

mockersf
Copy link
Member

Objective

  • Make example alien_cake_addict deterministic so that it's easier to check for regression

Solution

  • Use a seeded random

@mockersf mockersf added A-Build-System Related to build systems or continuous integration C-Examples An addition or correction to our examples labels Mar 16, 2024
@mockersf @rparrett
only in CI
4b594d0 
Co-Authored-By: Rob Parrett <robparrett@gmail.com>
@alice-i-cecile alice-i-cecile added this pull request to the merge queue Mar 17, 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 Mar 17, 2024
Merged via the queue into bevyengine:main with commit 1e1e11c Mar 17, 2024
26 checks passed
@james7132 james7132 mentioned this pull request Mar 17, 2024
Merged
@alice-i-cecile alice-i-cecile mentioned this pull request Mar 17, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rparrett rparrett rparrett left review comments

@alice-i-cecile alice-i-cecile alice-i-cecile approved these changes

@afonsolage afonsolage afonsolage approved these changes

Assignees
No one assigned
Labels
A-Build-System Related to build systems or continuous integration C-Examples An addition or correction to our examples 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
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@mockersf @rparrett @afonsolage @alice-i-cecile