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

Windows.Devices.Gpio POC #326

Closed
wants to merge 2 commits into from
Closed

Windows.Devices.Gpio POC #326

wants to merge 2 commits into from

Conversation

ghost
Copy link

@ghost ghost commented May 13, 2017

This is work in progress for evaluation only.
Goal here is to check if we are going the right way of doing things regarding file locations, methods and so on. Code is really incomplete and many methods do not contain any safety checks.

This PR also needs changes in nf-class-libraries projects.

Signed-off-by: Christophe Gerbier [email protected]

This is work in progress for evaluation only.
Goal here is to check if we are going the right way of doing things regarding file locations, methods and so on. Code is really incomplete and many methods do not contain any safety checks.

This PR also needs changes in nf-class-libraries projects.

Signed-off-by: Christophe Gerbier <[email protected]>
@nfbot
Copy link
Member

nfbot commented May 13, 2017

Hi @MikroBusNet,

I'm nanoFramework bot.
Thank you for your contribution!

A human will be reviewing it shortly. 😉

@josesimoes josesimoes assigned josesimoes and ghost May 14, 2017
@josesimoes josesimoes added Area: Interpreter Everything related with the interpreter, execution engine and such Area: Targets ⚠️ DO NOT MERGE ⚠️ Status: under review labels May 14, 2017
Reworked as discussed on the Slack channel

Signed-off-by: Christophe Gerbier <[email protected]>
@josesimoes
Copy link
Member

Closing. Replaced with #329

@josesimoes josesimoes closed this May 15, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: Interpreter Everything related with the interpreter, execution engine and such Area: Targets ⚠️ DO NOT MERGE ⚠️
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants