8000 refactor(lifecycle): improve component lifecycle documentation and sorting by Degfy · Pull Request #86 · gone-io/gone · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

refactor(lifecycle): improve component lifecycle documentation and sorting #86

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jun 8, 2025

Conversation

Degfy
Copy link
Member
@Degfy Degfy commented Jun 8, 2025

Add comprehensive documentation for lifecycle interfaces (BeforeStarter, AfterStarter, BeforeStopper, AfterStopper) with usage examples and best practices Fix typo in interface names (BeforeStoper/AfterStoper to BeforeStopper/AfterStopper) Add SortCoffins call in getByTypeAndPattern to ensure consistent ordering Enhance Application struct documentation with detailed explanations and analogies

…rting

Add comprehensive documentation for lifecycle interfaces (BeforeStarter, AfterStarter, BeforeStopper, AfterStopper) with usage examples and best practices
Fix typo in interface names (BeforeStoper/AfterStoper to BeforeStopper/AfterStopper)
Add SortCoffins call in getByTypeAndPattern to ensure consistent ordering
Enhance Application struct documentation with detailed explanations and analogies
Copy link
codecov bot commented Jun 8, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Files with missing lines Coverage Δ
application.go 100.00% <100.00%> (ø)
core_keeper.go 100.00% <100.00%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@Degfy Degfy merged commit 94a860e into main Jun 8, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant
0