Skip to content

Conversation

@davidwang001
Copy link

Reflection 1

@aculich
Copy link
Member

aculich commented Sep 7, 2013

Hi David- Can you format this markdown file according to the basic conventions here:

https://github.com/carwin/markdown-styleguide#basic-conventions-for-markdown-files

In particular, 80 character maximum for the line length. And break things into multiple
paragraphs.

That will make it easier for me to make comments using the github line commenting
system.

Also, you probably don't want to format each line by hand. If you use a text editor
like http://www.sublimetext.com/3 you will be able to have it format to 80 characters
for you so you don't have to do it by hand.

@aculich
Copy link
Member

aculich commented Sep 7, 2013

Once you re-format it, then submit a new pull request.

@davidwang001
Copy link
Author

Hey Aaron,

I downloaded sublime text 2, but I can't seem to find the line separation
tool even after doing a few Google searches. Is it possible on 2 or do I
need to get version 3? Sorry for the delay.

Thanks,
David

On Sat, Sep 7, 2013 at 4:09 PM, Aaron Culich notifications@github.comwrote:

Once you re-format it, then submit a new pull request.


Reply to this email directly or view it on GitHubhttps://github.com//pull/3#issuecomment-24011686
.

@aculich
Copy link
Member

aculich commented Sep 8, 2013

No worries about the delay-- in fact, your report about the roadblocks
you face is progress! I selected just a couple of you (those who sent
me pull requests) to update the formatting to conform to the conventions
so we can test out what works for just a couple of you before I ask the
whole class to do it!

If you're able to come to office hours from 1-3pm today (Sunday) then
we can take a look at it together.

@davidwang001
Copy link
Author

I actually wrote all my roadblocks down! Unfortunately I have an event
today that last the entire time, but I will try to make the Tuesday OH.
Also, if you are free tomorrow, I don't have class so just let me know.
Thanks again!

Best,
David

On Sun, Sep 8, 2013 at 12:45 PM, Aaron Culich notifications@github.comwrote:

No worries about the delay-- in fact, your report about the roadblocks
you face is progress! I selected just a couple of you (those who sent
me pull requests) to update the formatting to conform to the conventions
so we can test out what works for just a couple of you before I ask the
whole class to do it!

If you're able to come to office hours from 1-3pm today (Sunday) then
we can take a look at it together.


Reply to this email directly or view it on GitHubhttps://github.com//pull/3#issuecomment-24027875
.

Submission of reflection 2, still not quite sure if this will reach you
reflection 3 was put somewhere else accidentally, amended.
Added: "I am really glad that I have
a chance to code more, since that was one of the reasons I took this class in teh first place."
Sorry for turning in late, got distracted yesterday by unexpected visit from high school friends.
please read. *important*
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