Skip to content

Commit 822c766

Browse files
authored
fix(cli): [nan-1457] auto confirm for dry run (NangoHQ#2528)
## Describe your changes Honor the `--auto-confirm` option for dry run to automatically cycle through logs ## Issue ticket number and link NAN-1457 ## Checklist before requesting a review (skip if just adding/editing APIs & templates) - [ ] I added tests, otherwise the reason is: - [ ] I added observability, otherwise the reason is: - [ ] I added analytics, otherwise the reason is:
1 parent 7572d80 commit 822c766

File tree

2 files changed

+4
-3
lines changed

2 files changed

+4
-3
lines changed

.github/workflows/sync-integration-templates.yaml

+1
Original file line numberDiff line numberDiff line change
@@ -39,3 +39,4 @@ jobs:
3939
env:
4040
AWS_ACCESS_KEY_ID: ${{ secrets.AWS_ACCESS_KEY_ID }}
4141
AWS_SECRET_ACCESS_KEY: ${{ secrets.AWS_SECRET_ACCESS_KEY }}
42+
AWS_DEFAULT_REGION: ${{ secrets.AWS_REGION }}

packages/cli/lib/services/dryrun.service.ts

+3-3
Original file line numberDiff line numberDiff line change
@@ -368,9 +368,9 @@ export class DryRunService implements ScriptExecutorInterface {
368368

369369
while (index < logMessages.messages.length) {
370370
const remaining = logMessages.messages.length - index;
371-
const confirmation = await promptly.confirm(
372-
`There are ${remaining} logs messages remaining. Would you like to see the next 10 log messages? (y/n)`
373-
);
371+
const confirmation = options.autoConfirm
372+
? true
373+
: await promptly.confirm(`There are ${remaining} logs messages remaining. Would you like to see the next 10 log messages? (y/n)`);
374374
if (confirmation) {
375375
displayBatch();
376376
} else {

0 commit comments

Comments
 (0)