Skip to content

Conversation

@mohit7705
Copy link

What was done

  • Created an Apache Incubator podling logo for Hamilton using the existing Hamilton star logo, following the Apache press-kit guidelines
  • Added SVG (and PNG) assets under docs/_static
  • Applied the new incubator logo to the documentation homepage

Notes

  • Assets are prepared and ready for inclusion on the Apache logos page
  • No changes were made to the Apache Incubator ring or branding

Fixes #1435

@mohit7705
Copy link
Author

hi @skrawcz
can you verify in this

@skrawcz
Copy link
Contributor

skrawcz commented Jan 13, 2026

@mohit7705 nice. Question - can you make the star a bit bigger to fill the circle more? or is that spacing part of the guidelines?

@skrawcz
Copy link
Contributor

skrawcz commented Jan 19, 2026

@mohit7705 would love a minor tweak to what you did thanks.

Copy link
Contributor

Choose a reason for hiding this comment

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

I like this one more -- if the star could be a little bit bigger that would be great thanks

Copy link
Author

Choose a reason for hiding this comment

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

@skrawcz
I increased the size of the Hamilton star so it fills the circle better

Copilot AI review requested due to automatic review settings January 28, 2026 17:06
Copy link

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

Updates the documentation homepage to display the Apache Incubator podling logo for Hamilton, aligning the docs with Apache Incubator press-kit guidance (Fixes #1435).

Changes:

  • Added a centered incubator logo image to the docs homepage (docs/index.md).

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

Copy link

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

Copilot reviewed 1 out of 3 changed files in this pull request and generated no new comments.

Comments suppressed due to low confidence (1)

docs/index.md:27

  • The {figure} fenced directive isn’t closed before the {include} directive, so the {include} block is being parsed as part of the figure content. This will likely break the homepage rendering and/or Sphinx build. Close the figure fence (```) right after the caption, then start a new fenced block for {include}.
Apache Hamilton (Incubator Podling) logo

```{include} main.md

</details>



---

💡 <a href="/apache/hamilton/new/main/.github/instructions?filename=*.instructions.md" class="Link--inTextBlock" target="_blank" rel="noopener noreferrer">Add Copilot custom instructions</a> for smarter, more guided reviews. <a href="https://docs.github.com/en/copilot/customizing-copilot/adding-repository-custom-instructions-for-github-copilot" class="Link--inTextBlock" target="_blank" rel="noopener noreferrer">Learn how to get started</a>.

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.

Apply [press-kit materials]( https://incubator.apache.org/guides/press-kit.html) to docs & upload [to logos page](https://www.apache.org/logos/#).

2 participants