-
Notifications
You must be signed in to change notification settings - Fork 52
Open
Labels
✨ enhancementNew feature or requestNew feature or request
Description
The problem
When using --fail-on-changes and --pass-if-approved, the documented behaviour looks like it should be polling the build until it's approved, up until the timeout value, default 10 minutes.
What I'm seeing locally and in pipeline is the command exiting immediately after a build is completed.
e.g.
npx percy build:wait --fail-on-changes --pass-if-approved -b <build-id> -t 900000 -i 10000
[percy] Build #1013 finished! <url removed>
[percy] Found 1 changes
Script then exits. Happens with all our builds. If there's an issue with how i'm using the script then it might be more of a documentation issue than anything else but it certainly feels like a bug.
Effectively, wait isn't waiting.
Environment
- Node version: 21.5.0
@percy/cliversion: 1.28.9- Version of Percy SDK you’re using: 3.1.2
- If needed, a build or snapshot ID: N/A / All
- OS version: Windows 11 & the Docker node:lts-alpine image
- Type of shell command-line [interface]: Powershell + bash
Debug logs
npx percy build:wait --fail-on-changes --pass-if-approved -b 35202590 -t 900000 -i 10000 -v
[percy:config] Found config file: .percy.yml (0ms)
[percy:config] Using config:
{
version: 2,
snapshot: {
widths: [
1920
],
minHeight: 1280,
percyCSS: '',
enableJavaScript: false,
cliEnableJavaScript: true,
disableShadowDOM: false
},
discovery: {
networkIdleTimeout: 100
},
upload: {
files: '**/*.{png,jpg,jpeg}',
ignore: '',
stripExtensions: false
}
} (9ms)
[percy:env] Detected ci as null (9ms)
[percy:env] Detected commit as null (0ms)
[percy:env] Detected branch as null (0ms)
[percy:env] Detected git as <Removed Commit Information> (59ms)
[percy:client] Waiting for build 35202590... (27ms)
[percy:client] Get build 35202590 (0ms)
[percy:env] Detected info as null (0ms)
[percy:cli:build:wait] Build #1013 finished! https://percy.io/0599cc3c/Ziggy/builds/35202590 (470ms)
[percy:cli:build:wait] Found 1 changes (0ms)
Code to reproduce issue
N/A - Provided above.
Metadata
Metadata
Assignees
Labels
✨ enhancementNew feature or requestNew feature or request