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

HIL: remove 'static from sensors #1959

Merged
merged 1 commit into from Jun 19, 2020
Merged

HIL: remove 'static from sensors #1959

merged 1 commit into from Jun 19, 2020

Conversation

bradjc
Copy link
Contributor

@bradjc bradjc commented Jun 19, 2020

Pull Request Overview

Part of #1074. Removes 'static from clients in sensors HIL.

Testing Strategy

travis.

TODO or Help Wanted

n/a

Documentation Updated

  • Updated the relevant files in /docs, or no updates are required.

Formatting

  • Ran make prepush.

@bradjc bradjc added the HIL This affects a Tock HIL interface. label Jun 19, 2020
Copy link
Member

@ppannuto ppannuto left a comment

Choose a reason for hiding this comment

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

bors r+

Note: I am merging this "HIL changes" aggressively, as they aren't really changes to the semantics of the HIL, and they're simply implementing an approved RFC

@bors bors bot merged commit 198ff41 into master Jun 19, 2020
@bors bors bot deleted the hil-no-static-sensors branch June 19, 2020 19:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
HIL This affects a Tock HIL interface.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants