Skip to content

Conversation

@vsoch
Copy link
Member

@vsoch vsoch commented Apr 16, 2025

The headers do not seem to be searchable, so for a user to find the new link to flux-coral2 we need to add a little bit of description that includes the term. Since a user might also be looking for a system that is relevant, I added the primary and testing systems too. Here is what the search result looks like:

image

And the page:

image

I like this design better (with description) because many of the project names don't speak for themselves - we might consider adding brief descriptions to the others as well.

The headers do not seem to be searchable, so for
a user to find the new link to flux-coral2 we need
to add a little bit of description that includes
the term. Since a user might also be looking
for a system that is relevant, I added the primary
and testing systems too.

Signed-off-by: vsoch <[email protected]>
Copy link
Member

@jameshcorbett jameshcorbett left a comment

Choose a reason for hiding this comment

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

Looks great but some files were included accidentally it looks like?

@vsoch
Copy link
Member Author

vsoch commented Apr 17, 2025

I think we have those already:

https://github.com/flux-framework/flux-docs/tree/master/auto_examples

And when I built, the zip files were re-generated (with different timestamps?) I didn't do any git add or modifications to any other files aside from the projects one, and the clone was fresh.

@jameshcorbett
Copy link
Member

Yeah, those files are annoying. I don't really know what they are or how they're generated but I think if they're changed it should at least be a separate commit and give a reason why. I'd just drop them from the commit if I were you and ignore the issue for now. Otherwise worst case we might be changing them around all the time, in every PR.

@vsoch
Copy link
Member Author

vsoch commented Apr 17, 2025

It would be hard to decouple from an update because they are generated with make html. I think redoing the PR to get around a change here for a zip file date is excessive for a docs site.

@vsoch
Copy link
Member Author

vsoch commented Apr 17, 2025

When make html is run, it downloads new Python assets, so I suspect something changed in a dependency that updated those files. Notice that other auto generated files were not updated. We have to update the examples incrementally as they go into a static deployment, so those changes (in one changeset) would show up eventually. 🤷‍♀️

@jameshcorbett
Copy link
Member

Makes sense I'd say go ahead and set MWP!

@vsoch
Copy link
Member Author

vsoch commented Apr 17, 2025

Thanks @jameshcorbett - will do.

@vsoch vsoch added the merge-when-passing mark PR for auto-merging by mergify.io bot label Apr 17, 2025
@mergify mergify bot merged commit 6d99b1f into master Apr 17, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

merge-when-passing mark PR for auto-merging by mergify.io bot

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants