Skip to content

Conversation

@ndizazzo
Copy link
Owner

No description provided.

@codecov
Copy link

codecov bot commented Aug 11, 2025

Codecov Report

❌ Patch coverage is 71.05954% with 661 lines in your changes missing coverage. Please review.
⚠️ Please upload report for BASE (main@3445f2f). Learn more about missing BASE report.
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
action.go 64.54% 94 Missing and 12 partials ⚠️
actions/docker/docker_pull_action.go 47.82% 64 Missing and 8 partials ⚠️
actions/docker/docker_ps_action.go 61.20% 31 Missing and 14 partials ⚠️
actions/file/write_file_action.go 55.44% 42 Missing and 3 partials ⚠️
actions/docker/check_container_health_action.go 65.17% 34 Missing and 5 partials ⚠️
actions/docker/docker_image_list_action.go 62.50% 26 Missing and 10 partials ⚠️
actions/docker/docker_compose_ps_action.go 64.89% 23 Missing and 10 partials ⚠️
actions/file/read_file_action.go 51.02% 21 Missing and 3 partials ⚠️
actions/docker/docker_image_rm_action.go 75.78% 15 Missing and 8 partials ⚠️
actions/file/change_ownership_action.go 73.52% 12 Missing and 6 partials ⚠️
... and 20 more

❌ Your patch status has failed because the patch coverage (71.05%) is below the target coverage (80.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #10   +/-   ##
=======================================
  Coverage        ?   75.29%           
=======================================
  Files           ?       47           
  Lines           ?     6096           
  Branches        ?        0           
=======================================
  Hits            ?     4590           
  Misses          ?     1236           
  Partials        ?      270           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

- Migrate all actions from wrapper pattern to single struct with WithParameters()
- Add comprehensive parameter validation and type checking
- Implement support for static, action output, task output, and entity parameters
- Update all action constructors to use consistent parameter-aware pattern
- Add extensive test coverage for new parameter resolution system
- Fix failing tests and update backward compatibility expectations
- Update example tasks to demonstrate new parameter passing capabilities
@ndizazzo ndizazzo force-pushed the feature/parameter-passing branch from 7a0cb55 to 9d8372a Compare August 12, 2025 17:39
@ndizazzo ndizazzo force-pushed the feature/parameter-passing branch from efa49fd to 94537f3 Compare August 12, 2025 18:07
@ndizazzo ndizazzo merged commit 325df84 into main Aug 12, 2025
5 of 6 checks passed
@ndizazzo ndizazzo deleted the feature/parameter-passing branch August 12, 2025 18:11
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