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

Styling md anuj #59

Closed
wants to merge 2 commits into from
Closed

Conversation

anujji1999
Copy link
Contributor

@anujji1999 anujji1999 commented Apr 15, 2020

For #50

@lisa761
Copy link
Member

lisa761 commented Apr 15, 2020

Hi @anujji1999 You have updated your fork using a pull request. This causes a change in history of your master branch and our master branch. Merging this will duplicate commits in this repository. Reverting that commit will create another commit and change the history of your master branch even more. I suggest following this step by step to reset your master to ours and then make the relevant md styling changes in a new branch to create a new PR. Next time when you update your fork, please follow the steps given in the task instead to avoid this.

@lisa761
Copy link
Member

lisa761 commented Apr 15, 2020

As a general rule, remember that its okay for your master to be behind ours, but it should never be ahead of our master. You can check that on your fork repo page. It says your master is 1 commit ahead, 5 commits behind ACM-IGDTUW:Master.

@anujji1999 anujji1999 closed this Apr 15, 2020
@anujji1999 anujji1999 deleted the styling-md-anuj branch April 15, 2020 09:54
@anujji1999
Copy link
Contributor Author

anujji1999 commented Apr 15, 2020

#66
Is this fine?

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

Successfully merging this pull request may close these issues.

2 participants