{"action":"created","comment":{"url":"https://api.github.com/repos/SuperTux/wiki/pulls/comments/144721041","pull_request_review_id":69419258,"id":144721041,"diff_hunk":"@@ -1,42 +1,45 @@\n Before you start pushing your changes to SuperTux, you should read this. We can't force you to read it, but if you are at all new to git then you will almost certainly mess up at some point. This should help to avoid some mistakes that may be specific to Supertux.\n \n-## Do not use your master branch to make a pull request.\n-This will tie your master branch to the pull request. Note that you need a permission to access the Supertux master branch, so you have to create your own branch after that.\n-\n-1. Make sure that the remote origin is really the original SuperTux remote.\n- * Use `git remote -vv` to show your remotes. `git remote rename <old name> <new name>` to rename a remote.\n-2. `git branch master --set-upstream-to origin/master` or `git branch master -u origin/master`\n-3. `git pull`\n-\n-__When you want to change something, create a branch first__.\n-\n-1. Make sure that you are on the master branch by `git branch -vv`.\n-2. Create a branch by `git branch <branch name>`.\n-3. Switch to your newly created branch by `git checkout <branch name>`.\n-4. Change whatever you want and commit it.\n-5. Push it to your fork by `git push --set-upstream <remote>/<branch>` the first time you do this. After that push only by `git push`. The `<remote>` should be the name of your fork, usually your nickname. Use `git remote -vv` to get the names of all your remotes.\n-6. Make a pull request.\n-\n-## Don't merge branches, or pull without --rebase.\n-Merges create extra merge commits we don't need; rebase your changes instead. So when you want to make your pull request up-to-date with master, then don't use `git pull origin master`, but `git pull --rebase origin master` instead. After that use `git push -f` to push your changes to upstream.\n-\n-When you want to test someone's changes, then `git branch <branch name>` and `git branch <branch name> -u <remote>/<branch name>`. Make sure that you are on the root of that branch. Otherwise DO NOT push your merge commit!\n+Here are some of our recommendations for working with git:\n+\n+- **Do not use the `master` branch to make pull requests**\n+  - This will tie said branch to the pull request. It might happen that other stuff gets added to the SuperTux master branch while your pull request is under review. In that case, resolving conflicts to getting your master branch to work for your next pull request is hard.\n+- Make sure that the remote origin is really the main SuperTux remote\n+  1. Use `git remote -vv` to show your remotes. `git remote rename <old name> <new name>` to rename a remote\n+  2. `git branch master --set-upstream-to origin/master` or `git branch master -u origin/master`\n+  3. `git pull`\n+- **When you want to change something, create a branch first**\n+  1. Make sure that you are on the master branch by `git branch -vv`\n+  2. Create a branch by `git branch <branch name>`\n+  3. Switch to your newly created branch by `git checkout <branch name>`\n+  4. Change whatever you want and commit it\n+  5. Push it to your fork by `git push --set-upstream <remote>/<branch>` the first time you do this. After that push only by `git push`. The `<remote>` should be the name of your fork, usually your nickname. Use `git remote -vv` to get the names of all your remotes\n+  6. Make a pull request\n+- **Do not merge branches, or pull without --rebase**\n+  - Merges create extra merge commits we don't need and want in our main repo\n+  - Rebase your changes instead. So when you want to make your pull request up-to-date with master, use `git pull --rebase origin master` rather than a simple `git pull origin master`\n+  - If you rebased your changes, you will want to use `git push --force-with-lease` to push your changes to your fork\n+- When you want to test someone's changes\n+  - `git branch <branch name>` and `git branch <branch name> -u <remote>/<branch name>`\n+  - Make sure that you are on the root of that branch. Otherwise DO NOT push your merge commit!\n \n ## How to rebase your commits\n+\n Unless you have more than 20 commits in one pull request, don't squash normal commits down until asked. This makes it easier to check your code for problems if any occur. When it is agreed that your pull request is worth adding to the project, then you may be asked to rebase.\n \n For a tutorial on rebasing look [here](https://help.github.com/articles/about-git-rebase/).\n \n In order to get the rebase changes to take effect you will have to force push to your fork of supertux. **Only force push to your own fork of Supertux**.\n \n ## Other things you should respect\n-* Never force-push to a shared branch.\n-* Don't constantly ask other people to update their content.\n-* Use only constructive criticism, as being rude or acting annoyed could scare off a potential developer!\n-* Don't beg anyone to teach you a something. Instead, ask them for guidance if you encounter a problem when following an online tutorial\n-* Don't revert other people's work unless you're told to do so.\n-* Don't push erroneous commits to master.\n-* Don't make with vague suggestions. Always be specific of what you need.\n-* We don't need any visionaries. Being productive in suggestions those someone could do is dangerous here.\n-* When you make a big change, push it to your forked GitHub repo. and make a pull request.\n-* Owners: Try not to push more than 3 commits straight to master at once. For small changes it should be rebased first and for big changes it should be put in a pull request.\n\\ No newline at end of file\n+\n+- Never force-push to a shared branch.\n+- Don't constantly ask other people to update their content.\n+- Use only constructive criticism, as being rude or acting annoyed could scare off a potential developer!\n+- Don't beg anyone to teach you a something. Instead, ask them for guidance if you encounter a problem when following an online tutorial\n+- Don't revert other people's work unless you're told to do so.\n+- Don't push erroneous commits to master.\n+- Don't make with vague suggestions. Always be specific of what you need.\n+- We don't need any visionaries. Being productive in suggestions those someone could do is dangerous here.","path":"Git-guide,-TODO-'n'-notTODO.md","position":null,"original_position":75,"commit_id":"23d7c638e20971652fa6fde5ccedda8bd121764d","original_commit_id":"d28d82acb12e38c207dd14076ce7e268cb1b6523","user":{"login":"tobbi","id":583169,"avatar_url":"https://avatars0.githubusercontent.com/u/583169?v=4","gravatar_id":"","url":"https://api.github.com/users/tobbi","html_url":"https://github.com/tobbi","followers_url":"https://api.github.com/users/tobbi/followers","following_url":"https://api.github.com/users/tobbi/following{/other_user}","gists_url":"https://api.github.com/users/tobbi/gists{/gist_id}","starred_url":"https://api.github.com/users/tobbi/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tobbi/subscriptions","organizations_url":"https://api.github.com/users/tobbi/orgs","repos_url":"https://api.github.com/users/tobbi/repos","events_url":"https://api.github.com/users/tobbi/events{/privacy}","received_events_url":"https://api.github.com/users/tobbi/received_events","type":"User","site_admin":false},"body":"Strange sentence I don't understand.","created_at":"2017-10-15T14:17:09Z","updated_at":"2017-10-15T14:36:30Z","html_url":"https://github.com/SuperTux/wiki/pull/8#discussion_r144721041","pull_request_url":"https://api.github.com/repos/SuperTux/wiki/pulls/8","author_association":"OWNER","_links":{"self":{"href":"https://api.github.com/repos/SuperTux/wiki/pulls/comments/144721041"},"html":{"href":"https://github.com/SuperTux/wiki/pull/8#discussion_r144721041"},"pull_request":{"href":"https://api.github.com/repos/SuperTux/wiki/pulls/8"}}},"pull_request":{"url":"https://api.github.com/repos/SuperTux/wiki/pulls/8","id":146511187,"html_url":"https://github.com/SuperTux/wiki/pull/8","diff_url":"https://github.com/SuperTux/wiki/pull/8.diff","patch_url":"https://github.com/SuperTux/wiki/pull/8.patch","issue_url":"https://api.github.com/repos/SuperTux/wiki/issues/8","number":8,"state":"open","locked":false,"title":"Git guide: some restructuring and slight improvments","user":{"login":"maxteufel","id":6442985,"avatar_url":"https://avatars1.githubusercontent.com/u/6442985?v=4","gravatar_id":"","url":"https://api.github.com/users/maxteufel","html_url":"https://github.com/maxteufel","followers_url":"https://api.github.com/users/maxteufel/followers","following_url":"https://api.github.com/users/maxteufel/following{/other_user}","gists_url":"https://api.github.com/users/maxteufel/gists{/gist_id}","starred_url":"https://api.github.com/users/maxteufel/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/maxteufel/subscriptions","organizations_url":"https://api.github.com/users/maxteufel/orgs","repos_url":"https://api.github.com/users/maxteufel/repos","events_url":"https://api.github.com/users/maxteufel/events{/privacy}","received_events_url":"https://api.github.com/users/maxteufel/received_events","type":"User","site_admin":false},"body":"","created_at":"2017-10-13T17:24:42Z","updated_at":"2017-10-15T14:36:30Z","closed_at":null,"merged_at":null,"merge_commit_sha":"75273aa5d8cff98c3a61a2436a8ecda4ee3b4228","assignee":null,"assignees":[],"requested_reviewers":[],"milestone":null,"commits_url":"https://api.github.com/repos/SuperTux/wiki/pulls/8/commits","review_comments_url":"https://api.github.com/repos/SuperTux/wiki/pulls/8/comments","review_comment_url":"https://api.github.com/repos/SuperTux/wiki/pulls/comments{/number}","comments_url":"https://api.github.com/repos/SuperTux/wiki/issues/8/comments","statuses_url":"https://api.github.com/repos/SuperTux/wiki/statuses/23d7c638e20971652fa6fde5ccedda8bd121764d","head":{"label":"SuperTux:fix/git-guide-improvements","ref":"fix/git-guide-improvements","sha":"23d7c638e20971652fa6fde5ccedda8bd121764d","user":{"login":"SuperTux","id":6020189,"avatar_url":"https://avatars2.githubusercontent.com/u/6020189?v=4","gravatar_id":"","url":"https://api.github.com/users/SuperTux","html_url":"https://github.com/SuperTux","followers_url":"https://api.github.com/users/SuperTux/followers","following_url":"https://api.github.com/users/SuperTux/following{/other_user}","gists_url":"https://api.github.com/users/SuperTux/gists{/gist_id}","starred_url":"https://api.github.com/users/SuperTux/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/SuperTux/subscriptions","organizations_url":"https://api.github.com/users/SuperTux/orgs","repos_url":"https://api.github.com/users/SuperTux/repos","events_url":"https://api.github.com/users/SuperTux/events{/privacy}","received_events_url":"https://api.github.com/users/SuperTux/received_events","type":"Organization","site_admin":false},"repo":{"id":49335012,"name":"wiki","full_name":"SuperTux/wiki","owner":{"login":"SuperTux","id":6020189,"avatar_url":"https://avatars2.githubusercontent.com/u/6020189?v=4","gravatar_id":"","url":"https://api.github.com/users/SuperTux","html_url":"https://github.com/SuperTux","followers_url":"https://api.github.com/users/SuperTux/followers","following_url":"https://api.github.com/users/SuperTux/following{/other_user}","gists_url":"https://api.github.com/users/SuperTux/gists{/gist_id}","starred_url":"https://api.github.com/users/SuperTux/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/SuperTux/subscriptions","organizations_url":"https://api.github.com/users/SuperTux/orgs","repos_url":"https://api.github.com/users/SuperTux/repos","events_url":"https://api.github.com/users/SuperTux/events{/privacy}","received_events_url":"https://api.github.com/users/SuperTux/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/SuperTux/wiki","description":"SuperTux wiki content","fork":false,"url":"https://api.github.com/repos/SuperTux/wiki","forks_url":"https://api.github.com/repos/SuperTux/wiki/forks","keys_url":"https://api.github.com/repos/SuperTux/wiki/keys{/key_id}","collaborators_url":"https://api.github.com/repos/SuperTux/wiki/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/SuperTux/wiki/teams","hooks_url":"https://api.github.com/repos/SuperTux/wiki/hooks","issue_events_url":"https://api.github.com/repos/SuperTux/wiki/issues/events{/number}","events_url":"https://api.github.com/repos/SuperTux/wiki/events","assignees_url":"https://api.github.com/repos/SuperTux/wiki/assignees{/user}","branches_url":"https://api.github.com/repos/SuperTux/wiki/branches{/branch}","tags_url":"https://api.github.com/repos/SuperTux/wiki/tags","blobs_url":"https://api.github.com/repos/SuperTux/wiki/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/SuperTux/wiki/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/SuperTux/wiki/git/refs{/sha}","trees_url":"https://api.github.com/repos/SuperTux/wiki/git/trees{/sha}","statuses_url":"https://api.github.com/repos/SuperTux/wiki/statuses/{sha}","languages_url":"https://api.github.com/repos/SuperTux/wiki/languages","stargazers_url":"https://api.github.com/repos/SuperTux/wiki/stargazers","contributors_url":"https://api.github.com/repos/SuperTux/wiki/contributors","subscribers_url":"https://api.github.com/repos/SuperTux/wiki/subscribers","subscription_url":"https://api.github.com/repos/SuperTux/wiki/subscription","commits_url":"https://api.github.com/repos/SuperTux/wiki/commits{/sha}","git_commits_url":"https://api.github.com/repos/SuperTux/wiki/git/commits{/sha}","comments_url":"https://api.github.com/repos/SuperTux/wiki/comments{/number}","issue_comment_url":"https://api.github.com/repos/SuperTux/wiki/issues/comments{/number}","contents_url":"https://api.github.com/repos/SuperTux/wiki/contents/{+path}","compare_url":"https://api.github.com/repos/SuperTux/wiki/compare/{base}...{head}","merges_url":"https://api.github.com/repos/SuperTux/wiki/merges","archive_url":"https://api.github.com/repos/SuperTux/wiki/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/SuperTux/wiki/downloads","issues_url":"https://api.github.com/repos/SuperTux/wiki/issues{/number}","pulls_url":"https://api.github.com/repos/SuperTux/wiki/pulls{/number}","milestones_url":"https://api.github.com/repos/SuperTux/wiki/milestones{/number}","notifications_url":"https://api.github.com/repos/SuperTux/wiki/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/SuperTux/wiki/labels{/name}","releases_url":"https://api.github.com/repos/SuperTux/wiki/releases{/id}","deployments_url":"https://api.github.com/repos/SuperTux/wiki/deployments","created_at":"2016-01-09T17:57:29Z","updated_at":"2017-01-11T18:11:06Z","pushed_at":"2017-10-15T14:32:57Z","git_url":"git://github.com/SuperTux/wiki.git","ssh_url":"git@github.com:SuperTux/wiki.git","clone_url":"https://github.com/SuperTux/wiki.git","svn_url":"https://github.com/SuperTux/wiki","homepage":"https://github.com/SuperTux/supertux/wiki","size":113360,"stargazers_count":3,"watchers_count":3,"language":null,"has_issues":false,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":6,"mirror_url":null,"open_issues_count":3,"forks":6,"open_issues":3,"watchers":3,"default_branch":"master"}},"base":{"label":"SuperTux:master","ref":"master","sha":"d95efd68b3625a80c1969c790a30ab23dd2ea13a","user":{"login":"SuperTux","id":6020189,"avatar_url":"https://avatars2.githubusercontent.com/u/6020189?v=4","gravatar_id":"","url":"https://api.github.com/users/SuperTux","html_url":"https://github.com/SuperTux","followers_url":"https://api.github.com/users/SuperTux/followers","following_url":"https://api.github.com/users/SuperTux/following{/other_user}","gists_url":"https://api.github.com/users/SuperTux/gists{/gist_id}","starred_url":"https://api.github.com/users/SuperTux/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/SuperTux/subscriptions","organizations_url":"https://api.github.com/users/SuperTux/orgs","repos_url":"https://api.github.com/users/SuperTux/repos","events_url":"https://api.github.com/users/SuperTux/events{/privacy}","received_events_url":"https://api.github.com/users/SuperTux/received_events","type":"Organization","site_admin":false},"repo":{"id":49335012,"name":"wiki","full_name":"SuperTux/wiki","owner":{"login":"SuperTux","id":6020189,"avatar_url":"https://avatars2.githubusercontent.com/u/6020189?v=4","gravatar_id":"","url":"https://api.github.com/users/SuperTux","html_url":"https://github.com/SuperTux","followers_url":"https://api.github.com/users/SuperTux/followers","following_url":"https://api.github.com/users/SuperTux/following{/other_user}","gists_url":"https://api.github.com/users/SuperTux/gists{/gist_id}","starred_url":"https://api.github.com/users/SuperTux/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/SuperTux/subscriptions","organizations_url":"https://api.github.com/users/SuperTux/orgs","repos_url":"https://api.github.com/users/SuperTux/repos","events_url":"https://api.github.com/users/SuperTux/events{/privacy}","received_events_url":"https://api.github.com/users/SuperTux/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/SuperTux/wiki","description":"SuperTux wiki content","fork":false,"url":"https://api.github.com/repos/SuperTux/wiki","forks_url":"https://api.github.com/repos/SuperTux/wiki/forks","keys_url":"https://api.github.com/repos/SuperTux/wiki/keys{/key_id}","collaborators_url":"https://api.github.com/repos/SuperTux/wiki/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/SuperTux/wiki/teams","hooks_url":"https://api.github.com/repos/SuperTux/wiki/hooks","issue_events_url":"https://api.github.com/repos/SuperTux/wiki/issues/events{/number}","events_url":"https://api.github.com/repos/SuperTux/wiki/events","assignees_url":"https://api.github.com/repos/SuperTux/wiki/assignees{/user}","branches_url":"https://api.github.com/repos/SuperTux/wiki/branches{/branch}","tags_url":"https://api.github.com/repos/SuperTux/wiki/tags","blobs_url":"https://api.github.com/repos/SuperTux/wiki/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/SuperTux/wiki/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/SuperTux/wiki/git/refs{/sha}","trees_url":"https://api.github.com/repos/SuperTux/wiki/git/trees{/sha}","statuses_url":"https://api.github.com/repos/SuperTux/wiki/statuses/{sha}","languages_url":"https://api.github.com/repos/SuperTux/wiki/languages","stargazers_url":"https://api.github.com/repos/SuperTux/wiki/stargazers","contributors_url":"https://api.github.com/repos/SuperTux/wiki/contributors","subscribers_url":"https://api.github.com/repos/SuperTux/wiki/subscribers","subscription_url":"https://api.github.com/repos/SuperTux/wiki/subscription","commits_url":"https://api.github.com/repos/SuperTux/wiki/commits{/sha}","git_commits_url":"https://api.github.com/repos/SuperTux/wiki/git/commits{/sha}","comments_url":"https://api.github.com/repos/SuperTux/wiki/comments{/number}","issue_comment_url":"https://api.github.com/repos/SuperTux/wiki/issues/comments{/number}","contents_url":"https://api.github.com/repos/SuperTux/wiki/contents/{+path}","compare_url":"https://api.github.com/repos/SuperTux/wiki/compare/{base}...{head}","merges_url":"https://api.github.com/repos/SuperTux/wiki/merges","archive_url":"https://api.github.com/repos/SuperTux/wiki/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/SuperTux/wiki/downloads","issues_url":"https://api.github.com/repos/SuperTux/wiki/issues{/number}","pulls_url":"https://api.github.com/repos/SuperTux/wiki/pulls{/number}","milestones_url":"https://api.github.com/repos/SuperTux/wiki/milestones{/number}","notifications_url":"https://api.github.com/repos/SuperTux/wiki/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/SuperTux/wiki/labels{/name}","releases_url":"https://api.github.com/repos/SuperTux/wiki/releases{/id}","deployments_url":"https://api.github.com/repos/SuperTux/wiki/deployments","created_at":"2016-01-09T17:57:29Z","updated_at":"2017-01-11T18:11:06Z","pushed_at":"2017-10-15T14:32:57Z","git_url":"git://github.com/SuperTux/wiki.git","ssh_url":"git@github.com:SuperTux/wiki.git","clone_url":"https://github.com/SuperTux/wiki.git","svn_url":"https://github.com/SuperTux/wiki","homepage":"https://github.com/SuperTux/supertux/wiki","size":113360,"stargazers_count":3,"watchers_count":3,"language":null,"has_issues":false,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":6,"mirror_url":null,"open_issues_count":3,"forks":6,"open_issues":3,"watchers":3,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/SuperTux/wiki/pulls/8"},"html":{"href":"https://github.com/SuperTux/wiki/pull/8"},"issue":{"href":"https://api.github.com/repos/SuperTux/wiki/issues/8"},"comments":{"href":"https://api.github.com/repos/SuperTux/wiki/issues/8/comments"},"review_comments":{"href":"https://api.github.com/repos/SuperTux/wiki/pulls/8/comments"},"review_comment":{"href":"https://api.github.com/repos/SuperTux/wiki/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/SuperTux/wiki/pulls/8/commits"},"statuses":{"href":"https://api.github.com/repos/SuperTux/wiki/statuses/23d7c638e20971652fa6fde5ccedda8bd121764d"}},"author_association":"OWNER"}}
{"actor":{"display_login":"tobbi"}}