-
Notifications
You must be signed in to change notification settings - Fork 36
Conversation
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.
minor, otherwise good
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.
add some structure?
Adding headlines
Done |
Fix wording
Address wording comments
@smlng @waehlisch are you happy with the newest wording? |
"software provided with RIOT beyond the simple examples [within the RIOT |
Oops, I overlooked, that that comment also contained a wording change. I only focused on the reference. |
that's the reason why I copied more than the reference. |
@smlng ping? |
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.
ACK
Wasn't squashed.... |
damn, I'm not used to check this ... we definitely need more checks here. I suggest to cleanup and force-push on master, to me its not a problem here. |
This would remove the merge commit. Let's keep it as is. |
Provide README
Provides a README with some instructions (and the current build status of the repo).