Skip to content

Conversation

@mattiastefanelli
Copy link
Contributor

No description provided.

@emaclean emaclean self-requested a review October 8, 2025 09:54
Copy link
Contributor

@emaclean emaclean left a comment

Choose a reason for hiding this comment

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

Although I see the entry here in the text, when I checkout the branch and view the webpage the new entry doesn't seem to render - maybe just because of the space between the previous table entry and this one? Can you check out your branch and to run the webpage locally as described in the webpage here: https://pylhc.github.io/resources/wiki.html and try to edit and see if you can fix this Mattia.

Thanks,
Ewen

@emaclean emaclean requested a review from JoschD October 8, 2025 10:22
@JoschD
Copy link
Member

JoschD commented Oct 8, 2025

Haven't tested, but I also think it's the empty line.

It would also be the first entry this year to not have a tooltip in the table. On the other hand, as the title is quite straightforward, I don't think more information is needed. So take it as some optional input, but do as you prefer (unless Ewen has a strong preference)

@mattiastefanelli
Copy link
Contributor Author

Hey both, I should have fixed the MD entry. Most probably the problem was the empty line, I have tested it locally and it worked. I have add the tooltip as well with some lines taken from the MD summary.
Many thanks to both.

Copy link
Contributor

@emaclean emaclean left a comment

Choose a reason for hiding this comment

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

looks good - renders when I view locally.

@emaclean emaclean merged commit b50e0c2 into master Oct 16, 2025
2 checks passed
@fsoubelet fsoubelet deleted the mattiastefanelli-patch-1 branch October 16, 2025 15:30
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.

4 participants