Skip to content

Conversation

jutaro
Copy link
Contributor

@jutaro jutaro commented Jul 14, 2025

Improved Documentation UX

This PR improves the structure and metadata of the generated tracer documentation to enhance clarity and maintainability.

Changes

  • Simplified table of contents
    Only top-level tracers are now listed, improving readability and reducing noise in the generated index.

  • Footer generation localized to cardano-node
    Moved footer generation logic from trace-dispatcher into cardano-node, maintaining purity in the library and ensuring accurate, context-specific metadata.

  • Enhanced footer content
    The documentation footer now includes:

    • the cardano-node version
    • the current Git commit hash

@jutaro jutaro requested review from a team as code owners July 14, 2025 07:20
@jutaro jutaro force-pushed the jutaro/docu_generation2 branch from cf1fd48 to 21a06e0 Compare July 16, 2025 08:39
@jutaro jutaro force-pushed the jutaro/docu_generation2 branch from 4ebda3d to 24938d2 Compare July 17, 2025 06:06
@jutaro jutaro force-pushed the jutaro/docu_generation2 branch 2 times, most recently from ae16072 to a21760f Compare July 17, 2025 19:12
@jutaro jutaro force-pushed the jutaro/docu_generation2 branch from a21760f to 121826b Compare July 18, 2025 13:39
Copy link
Contributor

@mgmeier mgmeier left a comment

Choose a reason for hiding this comment

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

LGTM, thanks a lot @jutaro !

@mgmeier mgmeier enabled auto-merge July 18, 2025 13:41
@mgmeier mgmeier added this pull request to the merge queue Jul 18, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jul 18, 2025
@mgmeier mgmeier added this pull request to the merge queue Jul 18, 2025
Merged via the queue into master with commit 425b8c1 Jul 18, 2025
23 of 24 checks passed
@mgmeier mgmeier deleted the jutaro/docu_generation2 branch July 18, 2025 20:17
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