Skip to content

Conversation

@rezrah
Copy link
Contributor

@rezrah rezrah commented Jan 9, 2026

Updates the GitHub logos (invertocat + full) as part of the 2026 brand refresh.

Before After
Screenshot 2026-01-09 at 12 51 54 Screenshot 2026-01-09 at 12 51 44
Before After
Screenshot 2026-01-09 at 13 40 32 Screenshot 2026-01-09 at 13 39 32

Notes for reviewers:

  • Dimensions are the same for 16 and 24 versions

  • Paths in the updated logos sit differently within the live area. I've tried to optically match the previous padding and alignement to minimize regressions. It might need a quick review though @CameronFoxly

    E.g. of previous and new icons overlayed. Green is new.

    Screenshot 2026-01-09 at 13 13 37 Screenshot 2026-01-09 at 12 52 05
  • Other updates in this PR that aren't relevant to the logo change:

    • updated the LICENSE files to 2026
    • pinned Minitest to v5, as the v6 version that CI automatically uses doesn't include minitest/test anymore. Changes now make tests and CI pass ✅

Towards https://github.com/github/brand-marketing-design/issues/2674

@changeset-bot
Copy link

changeset-bot bot commented Jan 9, 2026

🦋 Changeset detected

Latest commit: 536e49e

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@primer/octicons Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@rezrah rezrah marked this pull request as ready for review January 9, 2026 15:47
@rezrah rezrah requested a review from a team as a code owner January 9, 2026 15:47
@rezrah rezrah requested review from Copilot and hectahertz January 9, 2026 15:47
Copy link
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.

Pull request overview

This PR updates GitHub brand logos (mark-github and logo-github) as part of a 2026 brand refresh. The changes include updating SVG path data for the new logo designs while maintaining the same dimensions.

Key changes:

  • Updated SVG paths for mark-github-16.svg, mark-github-24.svg, logo-github-16.svg, and logo-github-24.svg with new 2026 brand designs
  • Updated copyright year from 2025 to 2026 across all LICENSE files
  • Pinned minitest to version 5.x and added minitest/mock require for test compatibility

Reviewed changes

Copilot reviewed 10 out of 14 changed files in this pull request and generated no comments.

Show a summary per file
File Description
icons/mark-github-16.svg Updated GitHub mark logo (16px) with new SVG path for 2026 brand
icons/mark-github-24.svg Updated GitHub mark logo (24px) with new SVG path for 2026 brand
icons/logo-github-16.svg Updated GitHub full logo (16px) with new SVG path for 2026 brand
icons/logo-github-24.svg Updated GitHub full logo (24px) with new SVG path for 2026 brand
LICENSE Updated copyright year to 2026
lib/octicons_styled/LICENSE Updated copyright year to 2026
lib/octicons_react/LICENSE Updated copyright year to 2026
lib/octicons_node/LICENSE Updated copyright year to 2026
lib/octicons_jekyll/LICENSE Updated copyright year to 2026
lib/octicons_helper/LICENSE Updated copyright year to 2026
lib/octicons_gem/LICENSE Updated copyright year to 2026
lib/octicons_helper/test/helper.rb Added minitest/mock require for test compatibility
lib/octicons_helper/Gemfile Pinned minitest to ~> 5.0 for compatibility
.changeset/itchy-carrots-cover.md Added changelog entry for logo updates

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Contributor

@CameronFoxly CameronFoxly left a comment

Choose a reason for hiding this comment

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

LGTM from Brand 🚀 (should still get eyes from Primer eng, but just wanted to flag that Brand is in on this change, since it's a big one)


group :development, :test do
gem "minitest"
gem "minitest", "~> 5.0"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Pinning to v5 because CI loads in the newer v6, which no longer includes minitest/mock. This led to CI failures like this one, which are unrelated to my changes.

Choose a reason for hiding this comment

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

Yeah good catch, I'd rather have the major pinned anyway. Thanks!

Copy link

@hectahertz hectahertz left a comment

Choose a reason for hiding this comment

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

@hectahertz hectahertz merged commit 41c7805 into main Jan 13, 2026
18 checks passed
@hectahertz hectahertz deleted the rezrah/update-github-invertocat branch January 13, 2026 15:34
@primer primer bot mentioned this pull request Jan 13, 2026
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.

5 participants