Conversation
The base Squawk CLI defines some parameters which are not exposed via the action. This adds the CI relevant ones.
This removes the need for two Squawk calls.
|
Here's a PR with the old version: recipeyak/recipeyak#1485 Here's a PR with the new version: recipeyak/recipeyak#1484 I think we need to adjust how we find files |
|
Ohhhh, I think there was a regression introduced in a recent commit |
|
I think we may want to revert 4828f43 We can add proper pattern support inside the squawk project. That way we can fix this issue |
Agraphie
force-pushed
the
feature/some-squawk-params
branch
from
da53618 to
ae25674
Compare
|
Thanks for the tests. The problem is the leading or non-leading Natively supporting a pattern i.e. a list of files including patterns would be nice though, then we can save one parameter here |
|
@Agraphie Looks good! Thanks for the quick update! |
2 participants
The underlying Squawk CLI features many more parameters than the current action. This PR introduces multiple CI relevant parameters so it's easier to configure them when using this action.
It also simplifies the pattern and files usage into one Squawk call instead of two.