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.

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

Finally got around to set up my branch properly. #26

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

sdelmas
Copy link

@sdelmas sdelmas commented Sep 24, 2013

This adds the Cassandra parser. Feel free to use it in any way you see fit. No copyright from my side, it's all yours.

2) Adding local lib folder to ruby search path (may not be needed, but fixed load issues for me)
3) Updated default config to have Cassandra blocks, removed right hand info block and fixed order numbers.
@sukima
Copy link

sukima commented Feb 3, 2014

Pull Requests are a lot easier to manage and merge if the commit messages are formatted well.

http://git-scm.com/book/ch5-2.html - has some good advise on commit messages.

Simply: the first line is your subject and is <= 50 characters. Then a blank line and finally the body of the commit message (72 characters long). Here is an example:

Short (50 chars or less) summary of changes

More detailed explanatory text, if necessary.  Wrap it to about 72
characters or so.  In some contexts, the first line is treated as the
subject of an email and the rest of the text as the body.  The blank
line separating the summary from the body is critical (unless you omit
the body entirely); tools like rebase can get confused if you run the
two together.

Further paragraphs come after blank lines.

 - Bullet points are okay, too

 - Typically a hyphen or asterisk is used for the bullet, preceded by a
   single space, with blank lines in between, but conventions vary here

Pull Requests are formated the same way only the 72 character width for the body should be ignored (GitHub manually places breaks in their markdown unlike normal markdown).

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