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

[ci skip] Update Performance Testing guide. #7741

Merged
merged 1 commit into from Sep 24, 2012
Merged

Conversation

siong1987
Copy link
Contributor

  • remove Palmist which is no longer available.
  • update link to Query Reviewer.

- remove Palmist which is no longer available.
- update link to Query Reviewer.
vijaydev added a commit that referenced this pull request Sep 24, 2012
[ci skip] Update Performance Testing guide.
@vijaydev vijaydev merged commit 8a5eb2e into rails:master Sep 24, 2012
@vijaydev
Copy link
Member

@siong1987 please note that you can do such changes in docrails directly instead of sending in pull requests. Thanks.

@siong1987
Copy link
Contributor Author

@vijaydev got it. it is a bit confusing on the roles of both docrails and rails master. but, got it now. thanks.

@frodsan
Copy link
Contributor

frodsan commented Sep 24, 2012

@siong1987 you can check Contributing to Rails Documentation guide.

"If you're confident about your changes, you can push them directly yourself via docrails. Docrails is a branch with an open commit policy and public write access. Commits to docrails are still reviewed, but this happens after they are pushed. Docrails is merged with master regularly, so you are effectively editing the Ruby on Rails documentation."

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.

None yet

3 participants