Skip to content

STAGE 2: Embedded devices revisions #107

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

Closed
wants to merge 1 commit into from

Conversation

aturon
Copy link
Member

@aturon aturon commented Sep 30, 2018

Some organizational revisions and some sharpening of the text.

There are still a couple of placeholders that y'all need to flesh out:

  • The final item for the pitch
  • The looping video -- the plan here isn't entirely clear to me
  • Finalizing the production usage section

@aturon aturon added this to the Content: Stage 2 milestone Sep 30, 2018
@jamesmunns
Copy link
Member

Hey @aturon, I am currently working on this content in another PR: #106.

Would you prefer I merge those changes here? If so I can close that PR, and rebase my changes on yours.

@aturon
Copy link
Member Author

aturon commented Sep 30, 2018

@jamesmunns Hah, you beat me to it! I don't have a strong preference for how the PRs are structured -- and please take the revisions here as suggestions :-)

@jamesmunns jamesmunns mentioned this pull request Oct 1, 2018
9 tasks
@jamesmunns
Copy link
Member

Hey @aturon, I've integrated your text and organizational changes into PR #106. Could you please review that branch (and screenshots) for completeness?

The one change I left out was changing from 2 space indent to 4 space indent. If you'd like that to happen, I can make that in a commit by itself.

If you're alright with the changes there, I'd like to close this PR without merging.

@aturon
Copy link
Member Author

aturon commented Oct 3, 2018

Closing in favor of #106

@aturon aturon closed this Oct 3, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants