Conversation
Reviewer's guide (collapsed on small PRs)Reviewer's GuideAdds attribute tables for additional Sortable-related component types to the SortableLists sample page, likely as part of the 10.0.2 version bump, enabling their props/events to be documented in the demo UI. Class diagram for updated SortableLists sample pageclassDiagram
class SortableListsPage {
}
class AttributeTable {
Type type
}
class SortableList {
}
class SortableListItem {
}
class SortableOption {
}
class SortableEvent {
}
SortableListsPage --> AttributeTable : contains
AttributeTable --> SortableList : displays_attributes_for
AttributeTable --> SortableListItem : displays_attributes_for
AttributeTable --> SortableOption : displays_attributes_for
AttributeTable --> SortableEvent : displays_attributes_for
File-Level Changes
Assessment against linked issues
Possibly linked issues
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
There was a problem hiding this comment.
Pull request overview
This pull request bumps the BootstrapBlazor.Sortable package from version 10.0.1 to 10.0.2 and adds documentation tables for related types in the Sortable component demo page. The changes align with issue #7683, which requested adding SortableOption comments.
Changes:
- Updated BootstrapBlazor.Sortable package reference from 10.0.1 to 10.0.2
- Added AttributeTable components for SortableListItem, SortableOption, and SortableEvent types to provide comprehensive API documentation
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| src/BootstrapBlazor.Server/Components/Samples/SortableLists.razor | Added three AttributeTable components to document SortableListItem, SortableOption, and SortableEvent types |
| src/BootstrapBlazor.Server/BootstrapBlazor.Server.csproj | Bumped BootstrapBlazor.Sortable package version from 10.0.1 to 10.0.2 |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #7684 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 749 749
Lines 33190 33190
Branches 4603 4603
=========================================
Hits 33190 33190
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Link issues
fixes #7683
Summary By Copilot
Regression?
Risk
Verification
Packaging changes reviewed?
☑️ Self Check before Merge
Summary by Sourcery
Documentation: