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

docs: add @Vikingu-del as a contributor #291

Merged
merged 4 commits into from
May 30, 2024

Conversation

Vikingu-del
Copy link
Contributor

@Vikingu-del Vikingu-del commented May 8, 2024

Description

This PR adds @Vikingu-del as a contributor.

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

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

Related Issues

Closes #290

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)

Screenshot from 2024-05-08 12-30-25

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

@Vikingu-del Vikingu-del requested a review from a team May 8, 2024 10:44
Copy link

github-actions bot commented May 8, 2024

Congratulations on completing Chapter 5 of the Intro to OSS Course with your contribution to this repository @Vikingu-del! You're almost to the end of the course. Create a highlight of your contribution to our guestbook using the instructions in chapter 6 and share it with us!

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 @Vikingu-del,
Thank you for your PR! πŸ™Œ

However, please complete your PR form so we can understand and review the changes you made.

Can you please:

  • provide a description of the changes that you made in the "Description" section,
  • link your issue number in the "Related Issues" section. You can see your issue number right after the issue's title.

We also have merged some PRs before yours. Please resolve the conflicts before we can review and merge yours.

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

Thank you. πŸ™‚

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.

Hi @Vikingu-del,
We've helped you:

  • Resolve the merge conflicts in your PR. Check out this section about how to resolve conflicts for your future reference.
  • Complete the PR template by adding a description and the link to your issue in the "Related Issue" section. Please also check it out for your future contributions reference. ✨

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 4326b5d into open-sauced:main May 30, 2024
6 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: Getting used with Open-Sauced
2 participants