search

open-sauced / intro

Empowering Your Open Source Journey: From First Contribution to Project Leadership
https://opensauced.pizza/learn
Other
496 stars 96 forks source link

Fix: links, capitalization and wording adjustment in the "Tools to be Successful" chapter #132

Closed Haimantika closed 3 months ago

Haimantika commented 3 months ago

Description

This PR fixes links, capitalization and wording adjustment in the "Tools to be Successful" chapter.

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

Related Tickets & Documents

Fixes https://github.com/open-sauced/intro/issues/129

Mobile & Desktop Screenshots/Recordings

Steps to QA

Added to documentation?

[optional] Are there any post-deployment tasks we need to perform?

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

netlify[bot] commented 3 months ago

Deploy Preview for sauced-intro ready!

Name Link
Latest commit 544c901a80923614cf15c5d32be87742f0fb7bf8
Latest deploy log https://app.netlify.com/sites/sauced-intro/deploys/65f1e0796fe1f800088cec75
Deploy Preview https://deploy-preview-132--sauced-intro.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Haimantika commented 3 months ago

Hi @Haimantika

Thank you for working on this! ✨️ Before we review your PR, can you please:

  • Change the title of your PR with this format: 
    fix: things that changed
  • Fill in the description section with the details of your changes.

Completing PR form is necessary for maintainers to understand the changes that have been made in the PR.

And we have the guide about how to work with PR in our repos written in our contributing guidelines. Please take a moment to read it. 😊

Please let me know when you're done. Thank you!

Made the changes. Thanks for the comment. I missed these from the contributing guide.

adiati98 commented 3 months ago

Also, I see that you made changes directly on your main branch.

We won't merge PRs that's been working on main branch to avoid unexpected production fails.

For that reason, I'm closing this PR. 🙏

Please create a new PR and make sure that you create a new branch before working on it.

Thank you! 😊

adiati98 commented 3 months ago

Fixed in #133