Skip to content

process ops errors gracefully#6696

Merged
iHiD merged 11 commits intomainfrom
process-ops-errors-gracefully
Jan 19, 2024
Merged

process ops errors gracefully#6696
iHiD merged 11 commits intomainfrom
process-ops-errors-gracefully

Conversation

@ErikSchierboom
Copy link
Copy Markdown
Member

  • Process representation ops errors gracefully
  • Handle ops errors gracefully

@ErikSchierboom ErikSchierboom requested a review from iHiD as a code owner January 19, 2024 10:31
@ErikSchierboom ErikSchierboom force-pushed the process-ops-errors-gracefully branch from b98df4c to 05efcc4 Compare January 19, 2024 10:57
Comment thread app/commands/submission/analysis/process.rb Outdated
Comment thread app/commands/submission/analysis/process.rb Outdated
Comment thread app/commands/submission/representation/process.rb Outdated
Comment thread app/commands/submission/representation/process.rb Outdated
Comment thread app/commands/submission/representation/process.rb Outdated
ErikSchierboom and others added 8 commits January 19, 2024 14:19
Co-authored-by: Jeremy Walker <jez.walker@gmail.com>
Co-authored-by: Jeremy Walker <jez.walker@gmail.com>
Co-authored-by: Jeremy Walker <jez.walker@gmail.com>
Co-authored-by: Jeremy Walker <jez.walker@gmail.com>
Co-authored-by: Jeremy Walker <jez.walker@gmail.com>
@iHiD iHiD merged commit 01355bf into main Jan 19, 2024
@iHiD iHiD deleted the process-ops-errors-gracefully branch January 19, 2024 15:53
iHiD added a commit that referenced this pull request Feb 8, 2026
Fixes NoMethodError when tooling_job.execution_output is nil (e.g. ops
error with no output). Matches the existing guard in Analysis::Process
and Representation::Process added in PR #6696.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
iHiD added a commit that referenced this pull request Feb 8, 2026
Fixes NoMethodError when tooling_job.execution_output is nil (e.g. ops
error with no output). Matches the existing guard in Analysis::Process
and Representation::Process added in PR #6696.

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
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.

2 participants