Skip to content

fix: handle missing job logs - #1335

Open
kuntal1461 wants to merge 2 commits into
apify:masterfrom
kuntal1461:fix/output-job-log-missing-log
Open

fix: handle missing job logs#1335
kuntal1461 wants to merge 2 commits into
apify:masterfrom
kuntal1461:fix/output-job-log-missing-log

Conversation

@kuntal1461

@kuntal1461 kuntal1461 commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

Summary

Tests

  • CI=true pnpm exec vitest run test/local/lib/utils.test.ts
  • CI=true pnpm run test:local
  • CI=true pnpm run lint
  • CI=true pnpm run format
  • CI=true pnpm run build

@kuntal1461

Copy link
Copy Markdown
Contributor Author

Hi @l2ysho, @DaveHanns, @patrikbraborec 👋

Just wanted to kindly check if you'd have a chance to review this PR when possible.

I'm happy to make any changes or address any feedback.

Thank you for your time!

@l2ysho l2ysho left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thx @kuntal1461 🚀

@kuntal1461

Copy link
Copy Markdown
Contributor Author

Hi @l2ysho Happy to help! Let me know if anything else needs fixing. 🙌

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[ultra-review] outputJobLog crashes when a job log is missing

3 participants