Development#113
Merged
Merged
Conversation
New password input with visibility toggle, strength level indicator, and requirement validation. Includes i18n support via LayrzAppLocalizations. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Created 200 new tests across 13 test files (67 → 267 total tests) - Increased code coverage from 3.4% to 5.4% (10 → 27 files covered) Phase 1 - Pure Functions & Utilities (166 tests): - Color extensions: hex conversion, parsing, round-trip tests - Theme colors: all 18+ predefined themes - ORM error handling: storage, retrieval, i18n fallback - Colorblind filters: 6 accessibility filter types with strength interpolation - File utilities: MIME detection, base64 encoding, custom .lc extension - Separator extensions: SizedBox helpers (.w, .h, .wh) Phase 2 - Widget Components (34 tests): - Avatar widget: rendering, icons, name cleaning, tap handlers - Field error display: error messages, hiding, styling - Table controller: event listeners, sort/refresh events Testing Infrastructure: - Makefile with test/coverage/coverage-summary targets - Coverage report tool (tools/coverage_report.dart) - CI integration: tests run on PRs with result publishing - Updated .gitignore for Claude Code settings Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
- Parses test-results.json to extract pass/fail counts - Posts comment with test summary table (total, passed, failed, pass rate) - Updates existing comment on subsequent runs - Includes link to detailed check run for failures - Keeps detailed test results in Checks tab via dorny/test-reporter Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Configure Claude to automatically commit and push when user says: - "wrap it up" - "commit and push" - "summarize and commit" - "done, commit" Includes guidelines for proper commit message format and file staging. Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
❌ Test ResultsStatus: test(s) failed
🤖 Automated test results from checks workflow |
The auto-commit instructions are now in ~/.claude/custom_instructions.md and apply globally across all repositories. Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
- Run tests with --coverage in lint-dart job - Upload coverage artifact for reuse - Create separate coverage-comment job that downloads artifact - Remove duplicate test execution (runs only once now) - Use lcov for reliable coverage parsing - Post coverage as separate PR comment Benefits: - Tests run once instead of twice (faster) - Clean separation: test results in check run, coverage in comment - Follows artifact pattern from other projects Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
📊 Coverage Report
🤖 Coverage report from checks workflow |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.