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

system: run and return exit code on stop #411

Merged
merged 5 commits into from
Nov 15, 2021

Conversation

lucab
Copy link
Contributor

@lucab lucab commented Nov 5, 2021

Overview

This adds an additional running method to System, which allows
consumers to retrieve the stopping code.
Additionally, the existing run() is reworked to use that
internally, ensuring both codepaths are tested and aligned in
behavior.

PR Type

  • New feature
  • Refactor

PR Checklist

Check your PR fulfills the following:

  • Tests for the changes have been added / updated.
    • Test coverage inherited from existing ones, via the run() rework.
  • Documentation comments have been added / updated.
  • A changelog entry has been made for the appropriate packages.
  • Format code with the latest stable rustfmt

This adds an additional running method to `System`, which allows
consumers to retrieve the stopping code.
Additionally, the existing `run()` is reworked to use that
internally, ensuring both codepaths are tested and aligned in
behavior.
@robjtede robjtede self-requested a review November 15, 2021 02:58
@robjtede
Copy link
Member

robjtede commented Nov 15, 2021

change to match terminology: stop_with_code => run_with_code

@robjtede robjtede merged commit ce8ec15 into actix:master Nov 15, 2021
@lucab
Copy link
Contributor Author

lucab commented Nov 16, 2021

Thanks for the tweaks!

@lucab lucab deleted the ups/system-run-until-stop branch November 16, 2021 08:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants