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: gpio: remove 'static #1965

Merged
merged 1 commit into from Jul 20, 2020
Merged

hil: gpio: remove 'static #1965

merged 1 commit into from Jul 20, 2020

Conversation

bradjc
Copy link
Contributor

@bradjc bradjc commented Jun 20, 2020

Pull Request Overview

Part of #1074.

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 20, 2020
ppannuto
ppannuto previously approved these changes Jun 22, 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 d+

I think this one will unfortunately conflict with at least #1760 and any other new board / capsule PRs in flight, but there's probably never going to be a great window to get this in, and it's easy enough to add lifetimes to the other PRs

@bors
Copy link
Contributor

bors bot commented Jun 22, 2020

✌️ bradjc can now approve this pull request. To approve and merge a pull request, simply reply with bors r+. More detailed instructions are available here.

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+

@bors bors bot merged commit 93f799a into master Jul 20, 2020
@bors bors bot deleted the hil-no-static-gpio branch July 20, 2020 13:54
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

3 participants