Skip to content

Conversation

@tsurumi-yizhou
Copy link
Contributor

No description provided.

Signed-off-by: Don Yihtseu <i@yizhou.ac.cn>
@tsurumi-yizhou tsurumi-yizhou requested review from Copilot and pk5ls20 June 3, 2025 15:45
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR introduces internal documentation for various components of Lagrange.Core and updates the VitePress sidebar to surface those new pages.

  • Added markdown docs for Context, Events, Logic, Network, Packets, and Services under Lagrange.Core/Internal
  • Updated .vitepress/catalogue.ts to include an "Internal" section linking to each new doc

Reviewed Changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated no comments.

Show a summary per file
File Description
docs/v2/Lagrange.Core/Internal/Context.md Added high-level overview of Context module
docs/v2/Lagrange.Core/Internal/Events.md Added Protocol Events documentation
docs/v2/Lagrange.Core/Internal/Logic.md Added Logic layer description
docs/v2/Lagrange.Core/Internal/Network.md Added network utilities overview
docs/v2/Lagrange.Core/Internal/Packets.md Added packet encapsulation explanation
docs/v2/Lagrange.Core/Internal/Services.md Added Service abstraction note
docs/v2/.vitepress/catalogue.ts Added "Internal" sidebar grouping for new pages
Comments suppressed due to low confidence (2)

docs/v2/Lagrange.Core/Internal/Services.md:1

  • [nitpick] The heading "# Service" is singular, but the file is named Services.md and the sidebar entry uses "Services". Consider renaming the heading to # Services for consistency.
# Service

docs/v2/Lagrange.Core/Internal/Events.md:1

  • [nitpick] The sidebar link text is "Events" while the document heading is "Protocol Events". To avoid confusion, align the header with the link—e.g., use # Events or update the sidebar to "Protocol Events."
# Protocol Events

@pk5ls20 pk5ls20 merged commit f552266 into LagrangeDev:main Jun 4, 2025
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