This repository was archived by the owner on Mar 12, 2026. It is now read-only.
docs: Fix bugs identified during documentation migration#20
Closed
promptless[bot] wants to merge 1 commit intomainfrom
Closed
docs: Fix bugs identified during documentation migration#20promptless[bot] wants to merge 1 commit intomainfrom
promptless[bot] wants to merge 1 commit intomainfrom
Conversation
![promptless[bot]](https://avatars.githubusercontent.com/in/979718?s=60&v=4){kind=small}
| On Ubuntu runners, the action only supports headless browser mode. Chrome and Firefox contexts automatically fall back to headless mode when necessary. | ||
|
|
||
| If your tests require non-headless mode (like video recording with `startRecording`), use macOS or Windows runners instead: | ||
| If your tests require non-headless mode (like video recording with `record`), use macOS or Windows runners instead: |
Contributor
Author
There was a problem hiding this comment.
Citation: PR #185 comment by hawkeyexl noted this fix: "Fixed in 3fdfd56. Updated startRecording to record." The action was renamed from startRecording to record.
View source
|
|
||
| # Install dependencies | ||
| npm run setup | ||
| npm install |
Contributor
Author
There was a problem hiding this comment.
Citation: PR #185 comment by hawkeyexl noted this fix: "Fixed in 3fdfd56. Changed npm run setup to npm install." The setup script doesn't exist in docs/package.json.
View source
| maxVariation | number | Optional. Allowed variation in percentage of pixels between the new screenshot and the existing screenshot at `path`. If the difference between the new screenshot and the existing screenshot is greater than `maxVariation`, the step fails. If a screenshot doesn't exist at `path`, this value is ignored.<br/><br/>Minimum: 0. Maximum: 1 | `0.05` | ||
| overwrite | string | Optional. If `true`, overwrites the existing screenshot at `path` if it exists. | ||
| If `aboveVariation`, overwrites the existing screenshot at `path` if the difference between the new screenshot and the existing screenshot is greater than `maxVariation`.<br/><br/>Accepted values: `true`, `false`, `aboveVariation` | `aboveVariation` | ||
| overwrite | string | Optional. If `true`, overwrites the existing screenshot at `path` if it exists.<br/>If `aboveVariation`, overwrites the existing screenshot at `path` if the difference between the new screenshot and the existing screenshot is greater than `maxVariation`.<br/><br/>Accepted values: `true`, `false`, `aboveVariation` | `aboveVariation` |
Contributor
Author
There was a problem hiding this comment.
Citation: PR #185 comment noted the overwrite field description breaks the markdown table due to a newline splitting the row. Copilot suggested: "Keep the full description within the same table row (for example with <br/>)" and hawkeyexl confirmed: "Fixed in 3fdfd56."
View source
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 subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
Open this suggestion in Promptless to view citations and reasoning process
Applies bug fixes that were discovered and fixed during PR #185's documentation migration. These fixes were applied in the target repository but not backported to this source repo:
startRecordingtorecord(correct action name)npm run setuptonpm install(working setup command)overwritefield description was split across multiple linesTrigger Events
doc-detective/doc-detective PR #185: Migrate documentation and npmignore, upgrade fern version
Promptless Research (1 GitHub PR)
Agent Response
Tip: Configure how Promptless handles changelogs in Agent Settings 📋