Skip to content

fix: declaring dependencies of a stata script#110

Merged
tobiasraabe merged 4 commits into
mainfrom
codex/stata-task-dependencies
Jun 14, 2026
Merged

fix: declaring dependencies of a stata script#110
tobiasraabe merged 4 commits into
mainfrom
codex/stata-task-dependencies

Conversation

@tobiasraabe

@tobiasraabe tobiasraabe commented Jun 12, 2026

Copy link
Copy Markdown
Member

Summary

Closes #106

  • Allow generated Stata task runners to accept arbitrary dependency keyword arguments.
  • Add regression coverage for dependencies declared in the task signature and through @task(kwargs=...).
  • Clarify README examples for dependency/product tracking versus passing values to Stata through options.

Root Cause

pytask-stata collected normal pytask dependency arguments for Stata tasks, but the generated run_stata_script function only accepted the plugin's internal keyword arguments. During execution, pytask passed user dependency names to that runner, causing TypeError: run_stata_script() got an unexpected keyword argument ... before Stata was invoked.

@tobiasraabe tobiasraabe changed the title [codex] Fix Stata task dependencies fix: declaring dependencies of a stata script Jun 12, 2026
@tobiasraabe tobiasraabe marked this pull request as ready for review June 12, 2026 23:27
@tobiasraabe tobiasraabe enabled auto-merge June 14, 2026 10:25
@tobiasraabe tobiasraabe added this pull request to the merge queue Jun 14, 2026
Merged via the queue into main with commit 5cec22a Jun 14, 2026
22 checks passed
@tobiasraabe tobiasraabe deleted the codex/stata-task-dependencies branch June 14, 2026 10:27
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.

Error when adding dependency

1 participant