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

feat: add @asirialwis as a contributor #320

Merged
merged 5 commits into from
Jun 3, 2024

Conversation

asirialwis
Copy link
Contributor

@asirialwis asirialwis commented May 17, 2024

Description

This PR add @asirialwis as a contributor.

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

  • 🀝 Add a contributor
  • πŸ“ Documentation Update

Related Issues

Closes #322

Contributors Checklist

I've read through the Getting Started section.

  • βœ… Yes
  • ❌ Not yet

Have you run npm run contributors:generate to generate your profile and the badge on the README?

  • βœ… Yes
  • ❌ No

Added to documentation?

  • πŸ“œ README.md
  • πŸ™… no documentation needed

Screenshot (Required for PR Review)

Fosspng

[optional] What GIF best describes this PR or how it makes you feel?

@asirialwis asirialwis requested a review from a team May 17, 2024 16:19
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

PR Compliance Checks Passed!

Copy link
Member

@adiati98 adiati98 left a comment

Choose a reason for hiding this comment

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

Hey @asirialwis,
Thank you for your contribution! ✨

There are a few things here:

  • Please create an issue as mention in our course to accompany this PR as it's the best practice in open source.
  • You have a failed check that you need to address.
  • I see that you haven't completed your PR template and you've deleted a section here. That said, please put back the section and fill it in (you can look at previous PRs to see what you've removed) and complete your PR template so we can understand the changes you made and the background of this PR.

You can read this blog post to learn the importance of templates and how to fill in a PR template.

We're only reviewing this PR once we get all the information needed.
Please let us know once you've completed the template or if you have any question.

Thank you. πŸ™‚

@asirialwis asirialwis changed the title docs: add @asirialwis as a contributor feat: add @asirialwis as a contributor May 17, 2024
@asirialwis asirialwis requested a review from adiati98 May 17, 2024 18:00
@adiati98
Copy link
Member

Hey @asirialwis,

We have merged a PR before yours. Please resolve the conflicts before we can review your PR.

Do let us know if you need any help. Thanks. 😊

@asirialwis
Copy link
Contributor Author

I have solved the merge conflicts.

Copy link
Member

@adiati98 adiati98 left a comment

Choose a reason for hiding this comment

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

Hey there,

I highly recommend you read and follow the instructions to resolve merge conflicts in this repo.

From what I see here, you haven't run the command to generate your profile on README. And I think once you run it, it will throw an error. But don't get discouraged and just follow the instructions. πŸ™‚

Let me know if you have any questions or need any help. ✨️

Copy link
Member

@adiati98 adiati98 left a comment

Choose a reason for hiding this comment

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

Hi @asirialwis,
We've helped you resolve the merge conflicts in your PR. ✨

NOTE: For your future reference, please read and follow any instructions carefully. You can see the commit history in this PR to understand the changes that I made to resolve the conflicts.

If you haven't, you can join our community Discord. 😊
Also, check out our pizza-verse repo if you want to contribute further! πŸ•

pizza GIF

@adiati98 adiati98 merged commit 81e6e1b into open-sauced:main Jun 3, 2024
3 checks passed
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.

Feature: Add @asirialwis as a contributor
3 participants