Skip to content

Add anchors to community page#729

Merged
raulcd merged 1 commit intoapache:mainfrom
kou:community-anchor
Nov 18, 2025
Merged

Add anchors to community page#729
raulcd merged 1 commit intoapache:mainfrom
kou:community-anchor

Conversation

@kou
Copy link
Member

@kou kou commented Nov 17, 2025

We've changed Markdown backend to jekyll-commonmark from Kramdown by #673. Kramdown generates anchor for each headings but jekyll-commonmark doesn't. So we need to set anchors explicitly.

We need at least https://arrow.apache.org/community/#mailing-lists because it's used in our documents. But this PR adds anchors to all ## headings.

We've changed Markdown backend to jekyll-commonmark from Kramdown
by apache#673. Kramdown generates anchor for each headings but
jekyll-commonmark doesn't. So we need to set anchors explicitly.

We need at least https://arrow.apache.org/community/#mailing-lists
because it's used in our documents. But this PR adds anchors to all
`##` headings.
@github-actions
Copy link

Preview URL: https://kou.github.io/arrow-site

If the preview URL doesn't work, you may forget to configure your fork repository for preview.
See https://github.com/apache/arrow-site/blob/main/README.md#forks how to configure.

Copy link
Member

@raulcd raulcd left a comment

Choose a reason for hiding this comment

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

It does not seem to work on the GitHub pages for the PR:
https://kou.github.io/arrow-site/community/#stack-overflow

Am I testing this wrongly?

Maybe the deployed version isn't the correct one as I can't see the anchor on the inspector

@kou
Copy link
Member Author

kou commented Nov 18, 2025

Thanks for testing!

It seems that you test it before the preview is deployed:
https://github.com/kou/arrow-site/actions/runs/19426062916/job/55579018002 was done on 2025-11-17T11:36:18Z and
you tested on 2025-11-17T10:52:16Z.

https://kou.github.io/arrow-site/community/#stack-overflow works now. :-)

@raulcd
Copy link
Member

raulcd commented Nov 18, 2025

It seems that you test it before the preview is deployed:

I suppose I tested that it was indeed not working :)
Thanks @kou

@raulcd raulcd merged commit 42738fb into apache:main Nov 18, 2025
4 checks passed
@kou kou deleted the community-anchor branch November 18, 2025 08:21
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.

2 participants