Skip to content

Conversation

@alice-i-cecile
Copy link
Member

Ported from #182.

Status

  • review and revise
  • fix compilation issues

alice-i-cecile and others added 3 commits May 14, 2022 20:14
Co-authored-by: James Liu <contact@jamessliu.com>
@alice-i-cecile alice-i-cecile marked this pull request as ready for review May 15, 2022 00:56
.insert_resource(InputMode::Recording)
.add_system(record_secret)
// The system ordering here ensures that we don't spy on the input before it's entered
.add_system(check_secret.before(check_secret))
Copy link

Choose a reason for hiding this comment

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

Is this line correct? (I don't know Bevy but to me adding a system before itself seems like it doesn't make sense)

Copy link
Member Author

Choose a reason for hiding this comment

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

Nope, this is a typo :)

Copy link
Contributor

@IceSentry IceSentry left a comment

Choose a reason for hiding this comment

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

Tiny nitpick, otherwise I like it overall. Once the typo in the check_secret example is fixed this could probably be merged with the new-book and iterate when necessary.

}
```

In rare cases, you may need to add a resource after app startup.
Copy link
Contributor

Choose a reason for hiding this comment

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

I think I would extract that entire paragraph to it's own section, at least to make it easier to link to it.

rather than being automatically scheduled to the first available thread.
This can negatively impact system-level parallelism: systems with any nonsend parameters are always incompatible with each other as they both need to run on the main thread.

[`Send + Sync`]: https://doc.rust-lang.org/nomicon/send-and-sync.html
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm pretty sure markdownlint is only failing because this is missing a new line at the end of the file. I believe you can configure vscode to do that automatically.

@cart
Copy link
Member

cart commented Mar 5, 2023

Closing as we will very likely start new PRs (and use this as a reference) when we pick up the new book effort again.

@cart cart closed this Mar 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants