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.

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

unittests: adapt netdev_dummy for netdev event handlers #2166

Closed
wants to merge 7 commits into from
Closed

unittests: adapt netdev_dummy for netdev event handlers #2166

wants to merge 7 commits into from

Conversation

miri64
Copy link
Member

@miri64 miri64 commented Dec 8, 2014

unittest related follow-up to #2163.

@miri64 miri64 added Process: API change Integration Process: PR contains or issue proposes an API change. Should be handled with care. Area: tests Area: tests and testing framework Type: new feature The issue requests / The PR implemements a new feature for RIOT Area: network Area: Networking and removed Type: new feature The issue requests / The PR implemements a new feature for RIOT labels Dec 8, 2014
@miri64 miri64 added this to the Release 2014.12 milestone Dec 8, 2014
@miri64 miri64 added the State: waiting for other PR State: The PR requires another PR to be merged first label Dec 8, 2014
@phiros
Copy link
Member

phiros commented Dec 10, 2014

Looks good to me at first glance.
Will take a closer look at it as soon as the waiting for other PR label
goes away.

@miri64
Copy link
Member Author

miri64 commented Dec 10, 2014

You can make it go away, by reviewing #2163 ;-)

@miri64 miri64 force-pushed the unittests/fix/adapt-event-handler branch from 14ae9df to a476fef Compare December 16, 2014 16:16
@miri64
Copy link
Member Author

miri64 commented Dec 16, 2014

Rebased to #2163 and adapt for changes there

@OlegHahm OlegHahm modified the milestones: Release 2014.12, Release NEXT MAJOR Dec 18, 2014
@fnack fnack mentioned this pull request Jan 6, 2015
Closed
@OlegHahm OlegHahm modified the milestones: Release NEXT MAJOR, Network Stack Task Force Feb 6, 2015
@OlegHahm OlegHahm added NSTF and removed NSTF labels Feb 6, 2015
@miri64 miri64 modified the milestone: Network Stack Task Force Feb 8, 2015
@miri64
Copy link
Member Author

miri64 commented Feb 8, 2015

Abandoned, but memo'd for @haukepetersen's port according to #2278.

@miri64 miri64 closed this Feb 8, 2015
@miri64 miri64 deleted the unittests/fix/adapt-event-handler branch February 8, 2015 14:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@phiros phiros

Labels
Area: network Area: Networking Area: tests Area: tests and testing framework Process: API change Integration Process: PR contains or issue proposes an API change. Should be handled with care. State: waiting for other PR State: The PR requires another PR to be merged first
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@miri64 @phiros @OlegHahm