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

added maintiners in place of project admin #827

Merged
merged 5 commits into from
Mar 12, 2021

Conversation

haripriya9647
Copy link
Contributor

As this project is maintained by many folks soo projects admin is replaced by project maintainers.

Fixes #805

Replace issue_no with the issue number which is fixed in this PR

Type of change

Please delete options that are not relevant.

  • Readme Update

README.md Outdated Show resolved Hide resolved
Copy link
Collaborator

@vybhav72954 vybhav72954 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi there!!!

Even after your latest commit, I think they are stacked to the top to bottom

@vybhav72954
Copy link
Collaborator

I am pretty sure you can sort this out by using <table> tag

For more info, click here

Copy link
Collaborator

@vybhav72954 vybhav72954 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@vybhav72954 vybhav72954 requested a review from HarshCasper March 11, 2021 15:05
@vybhav72954 vybhav72954 added PR:Ready-to-Merge Ready to Merge PR:Reviewed PRs that have been reviewed. and removed Work-In-Progress labels Mar 11, 2021
Copy link
Owner

@HarshCasper HarshCasper left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Make an atomic commit by pushing all your commits into one single atomic commit. The overall diff is very small, so we don't need 5 commits for the same

@HarshCasper HarshCasper merged commit 74ee603 into HarshCasper:master Mar 12, 2021
@vybhav72954 vybhav72954 added GSSOC21 Level0 GSSoC Minor Documentation and removed PR:Ready-to-Merge Ready to Merge PR:Reviewed PRs that have been reviewed. labels Mar 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
GSSOC21 Level0 GSSoC Minor Documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove the Project Admin Portion from README
3 participants