Skip to content

Documentation Error #1077

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

Merged
merged 1 commit into from
Aug 7, 2012
Merged

Documentation Error #1077

merged 1 commit into from
Aug 7, 2012

Conversation

aebersold
Copy link
Contributor

missing parameter in laravel/documentation/views/html.md

@tobsn
Copy link
Contributor

tobsn commented Aug 7, 2012

commit from your patched branch to develop branch!

see: http://jasonlewis.me/blog/2012/06/how-to-contributing-to-a-github-project

@franzliedke
Copy link
Contributor

Probably okay in this case, though, right?

@sparksp
Copy link
Contributor

sparksp commented Aug 7, 2012

For a fix to the documentation like this, I'd say it's okay on master. For a change to code, especially adding or changing functionality then definitely develop.

sparksp added a commit that referenced this pull request Aug 7, 2012
Fix Documentation Error in html.md
@sparksp sparksp merged commit ebec6ba into laravel:master Aug 7, 2012
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.

4 participants