Skip to content

Conversation

@sammygrey
Copy link
Contributor

Added deprecation messages for Heartbeat, RenderStepped, and Stepped. Changed statements in PreRender, PreSimulation, and PostSimulation saying that they are equivalent to RenderStepped, Stepped, and Heartbeat to saying that they are their respective replacements.

Checks

By submitting your pull request for review, you agree to the following:

  • This contribution was created in whole or in part by me, and I have the right to submit it under the terms of this repository's open source licenses.
  • I understand and agree that this contribution and a record of it are public, maintained indefinitely, and may be redistributed under the terms of this repository's open source licenses.
  • To the best of my knowledge, all proposed changes are accurate.

Added deprecation messages for Heartbeat, RenderStepped, and Stepped. Changed statements in PreRender, PreSimulation, and PostSimulation saying that they are equivalent to RenderStepped, Stepped, and Heartbeat to saying that they are their respective replacements.
@sammygrey sammygrey requested a review from a team as a code owner March 3, 2025 19:58
@github-actions github-actions bot added engine reference Changes the Engine API Reference documentation changes requested This pull request has changes requested prior to merging labels Mar 3, 2025
Fixed a issue with quote formatting
@IgnisRBX
Copy link
Contributor

IgnisRBX commented Mar 4, 2025

Aside from the comments I made, I agree we can improve this document a bit by marking both RenderStepped and Stepped as deprecated, and point to their replacements.

sammygrey and others added 2 commits March 3, 2025 22:30
Removed deprecation message for Heartbeat (which was erroneously placed under UnbindFromRenderStep.
@IgnisRBX IgnisRBX merged commit 6bc3e72 into Roblox:main Mar 4, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changes requested This pull request has changes requested prior to merging engine reference Changes the Engine API Reference documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants