Skip to content

Remove broken Visual Git Cheatsheet link from Rock Paper Scissors lesson #29802

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

Conversation

adarshdotdev
Copy link

Because

The "Visual Git Cheatsheet" link in the Additional Resources section of the "Revisiting Rock Paper Scissors" lesson is broken and results in a privacy error. Removing this broken link will ensure all resources provided are accessible and up-to-date.

This PR

  • Removes the broken "Visual Git Cheatsheet" link from the Additional Resources section of the "Revisiting Rock Paper Scissors" lesson.

Issue

Closes #29796

Additional Information

No replacement resource was available for the broken link.

Pull Request Requirements

  • [ x] I have thoroughly read and understand The Odin Project curriculum contributing guide
  • [ x] The title of this PR follows the location of change: brief description of change format, e.g. Intro to HTML and CSS lesson: Fix link text
  • [ x] The Because section summarizes the reason for this PR
  • [ x] The This PR section has a bullet point list describing the changes in this PR
  • [ x] If this PR addresses an open issue, it is linked in the Issue section
  • [x ] If any lesson files are included in this PR, they have been previewed with the Markdown preview tool to ensure it is formatted correctly
  • [ x] If any lesson files are included in this PR, they follow the Layout Style Guide

@github-actions github-actions bot added the Content: Foundations Involves the Foundations content label Jun 25, 2025
@wise-king-sullyman wise-king-sullyman added the Status: Do Not Merge This PR should not be merged until further notice label Jun 26, 2025
@wise-king-sullyman
Copy link
Member

@adarshdotdev I appreciate you taking the time to make this PR, but since there was already an associated issue which someone else had already requested assignment on I'm going to wait for there response before reviewing this PR.

Again your contribution is much appreciated regardless of if this specific PR is merged or not, but in the future please don't take on issues without being assigned to them by a maintainer 🙂

@adarshdotdev
Copy link
Author

understood thank you

@wise-king-sullyman
Copy link
Member

Looks like this was resolved by #29813 so I'm going to close this PR, but thanks again for taking the time to make this PR!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Content: Foundations Involves the Foundations content Status: Do Not Merge This PR should not be merged until further notice
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Revisiting Rock Paper Scissors lesson: Broken Link in Additional Resources
2 participants