Skip to content
This repository has been archived by the owner on Sep 17, 2019. It is now read-only.

Implements #206 #236

Closed
wants to merge 2 commits into from
Closed

Implements #206 #236

wants to merge 2 commits into from

Conversation

narJH27
Copy link
Contributor

@narJH27 narJH27 commented Apr 11, 2017

Implements #206

Narayanan Seshan and others added 2 commits April 11, 2017 13:41
@narJH27
Copy link
Contributor Author

narJH27 commented Apr 11, 2017

@dbarrosop @mirceaulinic Good To merge?

@mirceaulinic
Copy link
Member

Hi @narJH27 - we either need to add this to #206, either add the documentation details from #206 here.
Anyway, as explained in #206, we cannot merge API changes before implementing the changes in the affected drivers.

@mirceaulinic
Copy link
Member

For more info see: #206 (comment) - there are quite a few drivers affected by this change.

@narJH27
Copy link
Contributor Author

narJH27 commented Apr 12, 2017

@mirceaulinic I understand. I will try to get to the other drivers when I can, but I do not have access to a lot of the vendor boxes. So if someone could help out, it would be great!

@mirceaulinic
Copy link
Member

Hi @narJH27 - I have moved your commit into #206 as that is the issue tracking the API change. Thanks!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@narJH27 @mirceaulinic