added tools list rendering#578
Conversation
✅ Deploy Preview for safety-critical-rust-consortium ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
fried-gluttony
left a comment
fried-gluttony
left a comment
There was a problem hiding this comment.
I added some comments
| "name": "flamegraph", | ||
| "type": "profiler", | ||
| "url": "https://github.com/flamegraph-rs/flamegraph", | ||
| "description": "Flame graph profiler for Rust", |
There was a problem hiding this comment.
Question/Note: as I understand flamegraph the host needs to be the target?
So I can't run my application on embedded target and to see the timing, as most safety-critical is embedded. How much does that really help - because I can't run my embedded/RTOS app on my host ?
| "title": "Rust Tooling for use in Safety Critical Projects", | ||
| "version": "1.1", | ||
| "date": "2025-09-29", | ||
| "tracked-standards": [ |
There was a problem hiding this comment.
The rendering of this tables seems off, because the open items squashes it.
Maybe put the Open items list below?
There was a problem hiding this comment.
Decided to move open topics to its own page
|
Since the docusaurus page can directly handle the available tools list if provided as JSON in the The details part is now also below a tool entry. For this, I switched from tables to a grid layout, but imo this looks much better. The description fields can now contain GH flavored Markdown, which makes it easier to add better readable context to tools. |
fried-gluttony
left a comment
fried-gluttony
left a comment
There was a problem hiding this comment.
I like the overview, great work :)
As mentioned during Tooling meeting, a table of contents would be cool - but I think is fine for now without
|
Great work, very clean overview of the tools! :-D I think it would be a great addition to have a table of contents for the various tool categories, so you can jump to later categories without having to scroll through all of the tools. Also, a minor improvement would be if we could add some forced line breaks ( |
|
I added the comments that I saw in the meeting notes. |
|
Fixed the display of standard levels to prevent wrapping, and added a "Tools Category" list at the start of the page to jump to the sections. @alexandruradovici this should resolve all open remarks that should be handled with this PR. |
Allows to use the list directly instead of pre-build step converting from YAML to JSON.
Switching to a grid layout instead of tables.
Detects URLs without the need for angle brackets.
1c98a43 to
d7c5baf
Compare
PLeVasseur
left a comment
PLeVasseur
left a comment
There was a problem hiding this comment.
This looks great!
Thank you to @manczak-ifx and @mhatzl for driving this forward :)
2814836
into
Safety-Critical-Rust-Consortium:main
6 participants
Please Review and comment