Skip to content

Commit 78b154d

Browse files
committed
docs: update TEST_COVERAGE_PLAN with accurate completion status
Corrected the description of getRemoteUrl tests to reflect that: - No default fallback was added (unnecessary - defaults are merged in prepareOptions) - Tests verify the function works when remote is provided from defaults - Clarified that prepareOptions() already merges defaults.remote='origin' This makes the plan accurately reflect the implementation rather than describing a removed fallback that was deemed unnecessary.
1 parent d626f80 commit 78b154d

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

TEST_COVERAGE_PLAN.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -173,10 +173,10 @@ Add `"types": "index.d.ts"` to package.json
173173
-Test: getRemoteUrl() returns correct URL from git config
174174
-Test: Error when not in a git repository
175175
-Test: Error when remote doesn't exist
176-
-Test: Works with minimal options (defaults to 'origin')
176+
-Test: Works when remote is provided from defaults
177177
-Test: Returns consistent URL for same remote
178178
-Test: Passes remote option to gh-pages getRemoteUrl method
179-
-Fixed: Added default 'origin' remote when undefined to prevent undefined string conversion
179+
-Note: No default fallback needed - prepareOptions() merges defaults.remote='origin'
180180

181181
**Why Critical:** This internal API could break in gh-pages v6+ upgrade
182182

0 commit comments

Comments
 (0)