Skip to content

add names to blog post#496

Merged
alessandrofelder merged 3 commits into
mainfrom
blog-author
May 20, 2026
Merged

add names to blog post#496
alessandrofelder merged 3 commits into
mainfrom
blog-author

Conversation

@PolarBean
Copy link
Copy Markdown
Member

closes #495
adds date and names to blog posts.
Before
image
After
image

@PolarBean PolarBean marked this pull request as draft May 19, 2026 15:55
@PolarBean PolarBean marked this pull request as ready for review May 19, 2026 15:59
Copy link
Copy Markdown
Member

@alessandrofelder alessandrofelder left a comment

Choose a reason for hiding this comment

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

I have built locally and it looks really nice 😍
One small suggestion about the newly introduced code.

I am confused why CI passes here and not on other PRs that we pushed to today, but that's for #494 😅

Comment thread docs/source/conf.py
linkcheck_ignore = [
# Sphinx-gallery output files only exist after a full HTML build, not during linkcheck
".*/*_examples/.*\.html",
r".*/*_examples/.*\.html",
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Out of interest, is this needed? (It seems unrelated to the PR contents, but may help with #494 ? Although looking at c4f1033 (where I tried to replicate this) it didn't help my PR 😅

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

this is just to fix this warning
image
it doesnt affect function

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

https://github.com/brainglobe/brainglobe.github.io/actions/runs/26103449429/job/76760688334#step:2:783 you can see the warning. Its not a big deal, just something small

Comment thread docs/source/conf.py Outdated
@adamltyson
Copy link
Copy Markdown
Member

Assuming this gets merged, could we add it to the other websites with blogs?

@PolarBean
Copy link
Copy Markdown
Member Author

yeah absolutely. ill merge this once the tests finish and open a pr on https://github.com/neuroinformatics-unit/neuroinformatics-unit.github.io. are there other blogs? maybe movement?

@adamltyson
Copy link
Copy Markdown
Member

adamltyson commented May 19, 2026

Yep, it's just NIU and movement, cc @niksirbi

@PolarBean
Copy link
Copy Markdown
Member Author

this only fails because gin is down, all good to merge?

@alessandrofelder
Copy link
Copy Markdown
Member

How do you know it fails because of GIN?
The CI is now giving the error documented in #494 - it is not directly related to GIN as far as I can tell (but it could be?)

@PolarBean
Copy link
Copy Markdown
Member Author

How do you know it fails because of GIN?

Screenshot_20260520_111437_Firefox I had assumed since it's a link check failure and it's complaining of a gin related time out. But maybe that's a faulty assumption

@alessandrofelder
Copy link
Copy Markdown
Member

Ah OK 💡 thanks, cool, I agree now 🐌 😅

@alessandrofelder alessandrofelder merged commit 6a43cc6 into main May 20, 2026
5 of 7 checks passed
@alessandrofelder alessandrofelder deleted the blog-author branch May 20, 2026 10:26
@niksirbi
Copy link
Copy Markdown
Member

I just found out there is an alternative way of doing this, see comment. Opinions welcome!

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.

[Feature] Blog posts dont mention the authors name on the actual page, only in the index

4 participants