Skip to content
This repository has been archived by the owner on Jan 5, 2024. It is now read-only.

enable-stack: true should be implied by other stack-related options #142

Closed
andreasabel opened this issue Dec 22, 2022 · 1 comment · May be fixed by haskell-actions/setup#6
Closed

enable-stack: true should be implied by other stack-related options #142

andreasabel opened this issue Dec 22, 2022 · 1 comment · May be fixed by haskell-actions/setup#6
Assignees
Labels
enhancement New feature or request re: inputs Concerning handling the inputs to this action re: stack Concerning CIs using stack
Milestone

Comments

@andreasabel
Copy link
Member

Currently, if you give stack-version but forget enable-stack: true, the action simply skips installation of stack.
A more user-friendly API would be to imply enable-stack: true if any of the other stack-related inputs is explicitly set by the user.

@andreasabel andreasabel added enhancement New feature or request re: stack Concerning CIs using stack labels Dec 22, 2022
@andreasabel andreasabel added the re: inputs Concerning handling the inputs to this action label Jan 5, 2023
@andreasabel andreasabel added this to the 3.0.0 milestone Jan 10, 2023
andreasabel added a commit to haskell-actions/setup that referenced this issue May 3, 2023
The options `enable-stack`, `stack-no-global`, `stack-setup-ghc` and `disable-matcher` are now proper booleans only accepting `true` or `false`.
Previously, they were true when set to some non-empty string, even when set to "false" or "off" etc.

`disable-matcher` is deprecated in favour of a new positive form `enable-matcher`.

`enable-stack` is now implied by setting another stack-related option, i.e., one of `stack-version`, `stack-no-global` and `stack-setup-ghc`.
Previously, it was a prerequisite to these options.

Contradictory options now give an error, such as `stack-no-global` with `ghc-version` or `enable-stack: false` with `stack-version`.

Fixes: haskell/actions#142
@andreasabel andreasabel self-assigned this May 4, 2023
@andreasabel
Copy link
Member Author

@andreasabel andreasabel modified the milestones: 3.0.0, 2.4.0 May 11, 2023
andreasabel added a commit to haskell-actions/setup that referenced this issue Jan 27, 2024
The options `enable-stack`, `stack-no-global`, `stack-setup-ghc` and `disable-matcher` are now proper booleans only accepting `true` or `false`.
Previously, they were true when set to some non-empty string, even when set to "false" or "off" etc.

`disable-matcher` is deprecated in favour of a new positive form `enable-matcher`.

`enable-stack` is now implied by setting another stack-related option, i.e., one of `stack-version`, `stack-no-global` and `stack-setup-ghc`.
Previously, it was a prerequisite to these options.

Contradictory options now give an error, such as `stack-no-global` with `ghc-version` or `enable-stack: false` with `stack-version`.

Fixes: haskell/actions#142
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request re: inputs Concerning handling the inputs to this action re: stack Concerning CIs using stack
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant