Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[GCI Task] Added ainesh to students.yml #162

Merged
merged 2 commits into from
Dec 4, 2016
Merged

[GCI Task] Added ainesh to students.yml #162

merged 2 commits into from
Dec 4, 2016

Conversation

champforevaa
Copy link
Contributor

@champforevaa champforevaa commented Dec 4, 2016

Please Merge the pull request

@champforevaa
Copy link
Contributor Author

Why isnt my profile showing on the main students.yml?

@hemantjadon
Copy link
Contributor

Hey @champforevaa
please squash your commits so we can merge this PR.

There are two ways in which you can squash.

  • While pushing another commit, you can amend to the previous commit. git commit --amend (when you already have a commit existing and you want to amend to it) instead of doing git commit -m "message" every time.

  • But then you dint follow the above amending rule, and you pushed multiple commits (like how you have done now) You can squash everything into one single commit. Check your git log and decide on till where you want to squash. suppose there are 2 commits of yours which you want to squash into single, then the command goes git rebase -i HEAD~2. Which leads to an editor where you will be asked for every commit whether you want to pick or squash. type squash for all the 2 commits present and exit. The rebase continues and you have force push the changes, as in git push -f origin <branch>.

This might be a bit confusing, but please watch this Video

Added Ainesh is student's section

Added ainesh to students.yml
Added Ainesh sootha to students.yml

Fixed Whitespace after the bio.

Fixed extra WHitespace above name
@champforevaa
Copy link
Contributor Author

Hi @hemantjadon
I have squashed the commits.

Now can the PR be merged?

Thanks!

@hemantjadon hemantjadon merged commit 501af77 into fossasia:gh-pages Dec 4, 2016
@hemantjadon
Copy link
Contributor

Submit task link so we can approve :)

# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants