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.

Sign up for GitHub

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

Jump to bottom

Fill in readme #31

Merged
merged 6 commits into from
Mar 24, 2013
Merged

Fill in readme #31

merged 6 commits into from
Mar 24, 2013

Conversation

whilp
Copy link
Contributor

@whilp whilp commented Mar 23, 2013

Seems legit?

obfuscurity
obfuscurity reviewed Mar 23, 2013
View reviewed changes
README.md
@@ -1,13 +1,30 @@
# Monitorama 2013 Hackathon

## Recommendations
Make something awesome at the Monitorama 2013 Hackathon. Join our speakers and 200 other hackers in a day of fixing, exploring, and building. Form a team or work on your own and tell us what you did at the end of the day. We'll award the best hacks with great prizes from Github and FitBit.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's mention the team size limits (up to 5).

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We also have prizes from Librato and AppFirst. Technically the prizes aren't "from Fitbit", we bought them. Also, the 'Bit' isn't capitalized.

@obfuscurity
Copy link
Member

I think that you want to use lists (bullets) for a lot of those.

@whilp
Copy link
Contributor Author

whilp commented Mar 23, 2013

Right on -- applying tweak sauce. Can you expand a bit on your last comment? Any place in particular that needs more cowbells bullets?

@obfuscurity
Copy link
Member

Well I don't see any bullets in those lists so I think they're going to wrap.

@whilp
Copy link
Contributor Author

whilp commented Mar 23, 2013

Hrm; looks fine to me.

@obfuscurity
Copy link
Member

Sorry, I was reading those -'s as a diff character instead rather than a bullet char. I typically use asterisks.

@whilp
Copy link
Contributor Author

whilp commented Mar 23, 2013

Gotcha; hadn't considered that ambiguity.

@obfuscurity
Copy link
Member

Getting this in so we can move forward.

obfuscurity added a commit that referenced this pull request Mar 24, 2013
@obfuscurity
Merge pull request #31 from monitorama/readme
7ec1953 
Fill in readme
@obfuscurity obfuscurity merged commit 7ec1953 into master Mar 24, 2013
@obfuscurity obfuscurity deleted the readme branch March 24, 2013 00:41
@whilp
Copy link
Contributor Author

whilp commented Mar 24, 2013

Fwiw, I had more commits locally that I couldn't push up because DDoS. I am
apply em here or on master after merge.

On Saturday, March 23, 2013, Jason Dixon wrote:

Getting this in so we can move forward.

—
Reply to this email directly or view it on GitHubhttps://github.com//pull/31#issuecomment-15349016
.

Will Maier
http://wcm.aier.us/

@obfuscurity
Copy link
Member

Shit, sorry about that.

On Sat, Mar 23, 2013 at 05:50:32PM -0700, Will Maier wrote:

Fwiw, I had more commits locally that I couldn't push up because DDoS. I am
apply em here or on master after merge.

On Saturday, March 23, 2013, Jason Dixon wrote:

Getting this in so we can move forward.

???
Reply to this email directly or view it on GitHubhttps://github.com//pull/31#issuecomment-15349016
.

Will Maier
http://wcm.aier.us/

Reply to this email directly or view it on GitHub:
#31 (comment)

Jason Dixon
http://obfuscurity.com/
https://twitter.com/obfuscurity

@obfuscurity obfuscurity restored the readme branch March 24, 2013 00:59
@obfuscurity
Copy link
Member

Restored the branch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
Boston 2013
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@whilp @obfuscurity