Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

revert: "feat: set usage arguments and flags as environment variables for toml tasks" #4413

Merged
merged 3 commits into from
Feb 16, 2025

Conversation

jdx
Copy link
Owner

@jdx jdx commented Feb 16, 2025

Reverts #4159

Fixes #4379

@jdx jdx changed the title Revert "feat: set usage arguments and flags as environment variables for toml tasks" revert: "feat: set usage arguments and flags as environment variables for toml tasks" Feb 16, 2025
@jdx
Copy link
Owner Author

jdx commented Feb 16, 2025

I added a test case to prevent this from regressing again @hverlin

@jdx jdx enabled auto-merge (squash) February 16, 2025 01:37
@jdx jdx merged commit 3bbb222 into main Feb 16, 2025
18 checks passed
@jdx jdx deleted the revert-4159-main branch February 16, 2025 01:54
jdx pushed a commit that referenced this pull request Feb 16, 2025
### 🐛 Bug Fixes

- properly replace non set flags with "false" by
[@IxDay](https://github.com/IxDay) in
[#4410](#4410)

### ◀️ Revert

- "feat: set usage arguments and flags as environment variables for toml
tasks" by [@jdx](https://github.com/jdx) in
[#4413](#4413)
@hverlin
Copy link
Contributor

hverlin commented Feb 16, 2025

Thank you!

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.

Using flag in run array causes error "unexpected word"
2 participants