Skip to content
This repository has been archived by the owner on May 6, 2019. It is now read-only.

Fixes: #8 : Added Me to Contributors #11

Merged
merged 5 commits into from Oct 1, 2018

Conversation

championpaddler
Copy link
Contributor

@championpaddler championpaddler commented Oct 1, 2018

Added Me to Contributors List.
Fixes: #8

@arshadkazmi42
Copy link
Owner

@sk9331657 Congrats on your first PR.

Small change needed here. Tag the issue number in PR description something like this #8

@arshadkazmi42 arshadkazmi42 added the Hacktoberfest For Hacktoberfest label Oct 1, 2018
@arshadkazmi42 arshadkazmi42 self-requested a review October 1, 2018 04:18
@championpaddler
Copy link
Contributor Author

@arshadkazmi42 Ok Sir.

@arshadkazmi42
Copy link
Owner

@sk9331657 you have still not tagged the issue number in your Pull Request description

@championpaddler championpaddler changed the title Added Me to Contributors Fixes: #12 : Added Me to Contributors Oct 1, 2018
@championpaddler
Copy link
Contributor Author

@arshadkazmi42 Now, it can be merged.

@arshadkazmi42
Copy link
Owner

@sk9331657 No need to create new issue, there is issue #8 already there, tag that issue. do not create new issue.

@championpaddler championpaddler changed the title Fixes: #12 : Added Me to Contributors Fixes: #8 : Added Me to Contributors Oct 1, 2018
Repository owner deleted a comment Oct 1, 2018
@championpaddler
Copy link
Contributor Author

@arshadkazmi42 Why Codacy is failing ??

Repository owner deleted a comment Oct 1, 2018
@arshadkazmi42
Copy link
Owner

@sk9331657 its indentation issue, I am ignoring it for now. I will be creating another issue to fix that.

@arshadkazmi42 arshadkazmi42 merged commit edec23c into arshadkazmi42:master Oct 1, 2018
@arshadkazmi42
Copy link
Owner

arshadkazmi42 commented Oct 1, 2018

Congrats @sk9331657 on landing your first PR

Repository owner deleted a comment Oct 1, 2018
@championpaddler
Copy link
Contributor Author

@arshadkazmi42 Thanks.

@sahildav sahildav mentioned this pull request Oct 1, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Hacktoberfest For Hacktoberfest
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants