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

Explicitly mention de facto styleguide #806

Merged
merged 1 commit into from Feb 10, 2013

Conversation

Projects
None yet
3 participants
@scribu
Contributor

scribu commented Feb 10, 2013

When new contributors want to hack on Jekyll, they have no way of knowing which coding standards are expected of their code, especially if they aren't experienced Ruby programmers.

This came out of a discussion with @parkr on #769.

parkr added a commit that referenced this pull request Feb 10, 2013

Merge pull request #806 from scribu/patch-1
Explicitly mention defacto styleguide

@parkr parkr merged commit 78c3718 into jekyll:master Feb 10, 2013

1 check passed

default The Travis build passed
Details

@scribu scribu deleted the scribu:patch-1 branch Feb 10, 2013

parkr added a commit that referenced this pull request Feb 10, 2013

@parkr

This comment has been minimized.

Member

parkr commented Feb 10, 2013

Thanks dude! Sorry for not being more explicit previously.

@scribu

This comment has been minimized.

Contributor

scribu commented Feb 10, 2013

No worries.

@jekyll jekyll locked and limited conversation to collaborators Feb 27, 2017

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.