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

add: profile #1246

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

add: profile #1246

wants to merge 2 commits into from

Conversation

hi-ammad
Copy link

@hi-ammad hi-ammad commented Sep 15, 2024

What type of PR is this? (check all applicable)

  • πŸš€ Added Name
  • ✨ Feature
  • βœ… Joined Community
  • [βœ…] 🌟 ed the repo
  • πŸ› Grammatical Error
  • πŸ“ Documentation Update
  • 🚩 Other

Description

Add Link of GitHub Profile

https://github.com/hi-ammad

Copy link

welcome bot commented Sep 15, 2024

Thanks for opening this pull request! Do join our discord server DevSpace, a place for Awesome Developer's :)

README.md Outdated
@@ -25,33 +25,35 @@
</div>

### Contents:
- [Categories](#categories)

Choose a reason for hiding this comment

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

Why did you refactor the code? Was wrong?

Copy link
Author

Choose a reason for hiding this comment

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

i dont remember to be honest , maybe i faced some issue, i want to contribute , accept my pull request please thanks.

Choose a reason for hiding this comment

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

But if you want to add just one profile, should be changed just one line. Refactor should be in other commit, even PR.

@hi-ammad
Copy link
Author

i fixed it, check it now.

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

Successfully merging this pull request may close these issues.

2 participants