Skip to content

Fix up PR 4826 without stray commits#4875

Merged
ahpook merged 4 commits intomainfrom
ahpook/fixup-4826
Sep 7, 2025
Merged

Fix up PR 4826 without stray commits#4875
ahpook merged 4 commits intomainfrom
ahpook/fixup-4826

Conversation

@ahpook
Copy link
Copy Markdown
Contributor

@ahpook ahpook commented Sep 7, 2025

There are a bunch of stray commits in the original PR, so this just pulls the logo over while retaining attribution from the original contributor

  • Add files via upload
  • Fix erroneously named file

Closes #4826

Please confirm this pull request meets the following requirements:

Which change are you proposing?

  • Suggesting edits to an existing topic or collection
  • Curating a new topic or collection
  • Something that does not neatly fit into the binary options above

Editing an existing topic or collection

I'm suggesting these edits to an existing topic or collection:

  • Image (and my file is *.png, square, dimensions 288x288, size <= 75 kB)
  • Content (and my changes are in index.md)

Please replace this line with an explanation of why you think these changes should be made.

Curating a new topic or collection

  • I've formatted my changes as a new folder directory, named for the topic or collection as it appears in the URL on GitHub (e.g. https://github.com/topics/[NAME] or https://github.com/collections/[NAME])
  • My folder contains a *.png image (if applicable) and index.md
  • All required fields in my index.md conform to the Style Guide and API docs: https://github.com/github/explore/tree/main/docs

Please replace this line with an explanation of why you think this topic or collection should be curated.

Something that does not neatly fit into the binary options above

  • My suggested edits are not about an existing topic or collection, or at least not a single one
  • My suggested edits are not about curating a new topic or collection, or at least not a single one
  • My suggested edits conform to the Style Guide and API docs: https://github.com/github/explore/tree/main/docs

Please replace this line with an explanation of your proposed changes.


Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.

piratabrasileiro1 and others added 2 commits September 7, 2025 10:43
Signed-off-by: Gabriel Lima Lopes <135924740+piratabrasileiro1@users.noreply.github.com>
@ahpook ahpook requested a review from a team as a code owner September 7, 2025 17:45
Copilot AI review requested due to automatic review settings September 7, 2025 17:45
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Missing `"` at the end of the string causes workflow failures.

I introduced this in 8d7fa6d but
I guess it never caused a problem till now...?
@ahpook ahpook merged commit b136f7f into main Sep 7, 2025
7 of 8 checks passed
@ahpook ahpook deleted the ahpook/fixup-4826 branch September 7, 2025 17:59
@piratabrasileiro1
Copy link
Copy Markdown
Contributor

Thanks

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.

3 participants