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 a GIF to footbal & cars category #11

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

Conversation

elvis-kimutai
Copy link

football and cars GIFs

Copy link
Owner

@Cli4d Cli4d left a comment

Choose a reason for hiding this comment

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

Hi @elvis-kimutai

Thank you for attending the workshop on Open Source. Apologies for it taking me this long to close your pull request. The pull request Looks Good To Me (LGTM) ✅. One feedback would be for you to always add a description to your pull request. It helps give more context into why you did what you did.
Continue with your journey of Open Source through the following resources

  • GitHub Skills: You can learn more about GitHub including working with GitHub CodeSpaces and collaborating with GitHub
  • Microsoft Learn GitHub paths: Learn more beginner and advanced concepts on Git and GitHub using Microsoft Learn
  • Good commits article: Learn how to write good commit messages that communicate the right message
  • Branching workflows: Learn about the different branching workflows for your projects and even when working with open source projects
  • Join Hacktoberfest: Join the biggest open source movements with thousands of open source projects to contribute to.

You can access the workshop's slides here which has more resources.

Enjoy your journey in open source!

@Cli4d Cli4d added automerge Trigger GA to automerge PR and removed automerge Trigger GA to automerge PR labels Jan 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge Trigger GA to automerge PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants