-
Notifications
You must be signed in to change notification settings - Fork 594
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
docs: how to control GPIO pins within rootless Docker container on Ra… #2276
Conversation
…spberry Pi Relates to dotnet#2169
@dotnet-policy-service agree |
azp /run |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, this is really useful for people in the same situation as you. I made couple of suggestions while reading to have you gaining time with the linter ;-)
Otherwise, as already exchange on the topic in the issue, things look clear to me.
Co-authored-by: Laurent Ellerbach <[email protected]>
Co-authored-by: Laurent Ellerbach <[email protected]>
Co-authored-by: Laurent Ellerbach <[email protected]>
Co-authored-by: Laurent Ellerbach <[email protected]>
Co-authored-by: Laurent Ellerbach <[email protected]>
Co-authored-by: Laurent Ellerbach <[email protected]>
Co-authored-by: Laurent Ellerbach <[email protected]>
Co-authored-by: Laurent Ellerbach <[email protected]>
Co-authored-by: Laurent Ellerbach <[email protected]>
Co-authored-by: Laurent Ellerbach <[email protected]>
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please check the linter result and check if anything is left!
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
Is there anything that I can do regarding the failed builds? |
From the build, I can see 1 broken link in one of the file. Can you just remove it as it doesn't really seems to exist anymore? Bonus point if you find a replacement link. Then good to go! |
/azp run |
Commenter does not have sufficient privileges for PR 2276 in repo dotnet/iot |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
Is there anything I can help you with regarding the failing builds, @Ellerbach ? |
No, it's just failing for an unknown reason. Let's see if we merge it like this as all is correct. We do have a triage call on Thursday. |
/azp run |
Pull request contains merge conflicts. |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
Relates to #2169
Dear @Ellerbach,
I kindly ask you for your feedback regarding the new page "Control GPIO pins within rootless Docker container on Raspberry Pi".
Since I'm not a native English speaker, I also appreciate feedback regarding certain sentences/words that might sound "bumpy" 🙂
Last but not least, maybe we can consider whether it's worth asking somebody from the "rootless Docker .NET" world like Rich Landers for his feedback 🤔 just to be on the safe side that I'm not writing anything wrong.
Microsoft Reviewers: Open in CodeFlow