Skip to content

Joining the embedded hal team #220

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Sep 29, 2018
Merged

Joining the embedded hal team #220

merged 1 commit into from
Sep 29, 2018

Conversation

ryankurte
Copy link
Contributor

Hey all, I'd like to join the embedded hal team to help triage and work through the issue pile ^_^

@ryankurte
Joining the embedded hal team
9df8f74 
Hey all, I'd like to join the embedded hal team to help triage and get through the issue pile ^_^
@ryankurte ryankurte requested review from dylanmckay, jcsoo and a team as code owners September 28, 2018 20:16
@japaric
Copy link
Member

japaric commented Sep 28, 2018

r? @rust-embedded/hal

japaric
japaric approved these changes Sep 28, 2018
View reviewed changes
Copy link
Member

@japaric japaric left a comment

Choose a reason for hiding this comment

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

🎉

therealprof
therealprof approved these changes Sep 28, 2018
View reviewed changes
Copy link
Contributor

@therealprof therealprof left a comment

Choose a reason for hiding this comment

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

Could definitely use more hands here.

hannobraun
hannobraun approved these changes Sep 28, 2018
View reviewed changes
Copy link
Member

@hannobraun hannobraun left a comment

Choose a reason for hiding this comment

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

And that makes a majority :)

Welcome @ryankurte!

@japaric japaric added decision-accepted We voted on this proposal and accepted it and removed S-waiting-on-team labels Sep 29, 2018
@japaric
Copy link
Member

japaric commented Sep 29, 2018

Welcome to the HAL team, @ryankurte!

bors r+

bors bot added a commit that referenced this pull request Sep 29, 2018
@bors @ryankurte
Merge #220
77a86ea 
220: Joining the embedded hal team r=japaric a=ryankurte

Hey all, I'd like to join the embedded hal team to help triage and work through the issue pile ^_^

Co-authored-by: Ryan <ryankurte@users.noreply.github.com>
@bors
Copy link
Contributor

bors bot commented Sep 29, 2018

Build succeeded

@bors bors bot merged commit 9df8f74 into master Sep 29, 2018
@bors bors bot deleted the ryankurte-embedded-hal-team branch September 29, 2018 18:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hannobraun hannobraun hannobraun approved these changes

@thejpster thejpster thejpster approved these changes

@therealprof therealprof therealprof approved these changes

@japaric japaric japaric approved these changes

@paoloteti paoloteti paoloteti approved these changes

@ithinuel ithinuel Awaiting requested review from ithinuel

@dylanmckay dylanmckay Awaiting requested review from dylanmckay

@jcsoo jcsoo Awaiting requested review from jcsoo

Assignees
No one assigned
Labels
decision-accepted We voted on this proposal and accepted it T-hal
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@ryankurte @japaric @hannobraun @thejpster @therealprof @paoloteti