Skip to content

🚀 Improve error handling and remove unused parameters#1192

Merged
dsyme merged 2 commits intomainfrom
noerr
Oct 3, 2025
Merged

🚀 Improve error handling and remove unused parameters#1192
dsyme merged 2 commits intomainfrom
noerr

Conversation

@dsyme
Copy link
Copy Markdown
Collaborator

@dsyme dsyme commented Oct 3, 2025

Summary

  • Modified workflow log validation to continue on errors instead of failing
  • Removed unused parameters in various function signatures
  • Simplified code by removing unnecessary arguments

Key Changes

  • Changed GitHub workflow error validation to log errors instead of failing the workflow step
  • Removed unnecessary userSecretName parameter in addEngineSecret function
  • Removed verbose parameter from generateCacheMemorySteps function
  • Simplified function signatures in multiple engine and workflow configuration methods

@dsyme dsyme merged commit e37047b into main Oct 3, 2025
19 checks passed
@dsyme dsyme deleted the noerr branch October 3, 2025 13:20
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.

1 participant