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

updating with force-all-dirs #34

Closed
wants to merge 1 commit into from
Closed

updating with force-all-dirs #34

wants to merge 1 commit into from

Conversation

breaman
Copy link

@breaman breaman commented Nov 17, 2021

Since tfsec does not walk the hierarchy like it used to, this piece should fix the issue that has been reported for issue:

#33

@skuethe
Copy link

skuethe commented Nov 25, 2021

There is already an open PR #32 which gives more flexibility with specifying the parameters you want on workflow definition level.

In addition there is a bug when providing this parameter - see https://github.com/aquasecurity/tfsec/issues/1129
So it should not be forced.

@owenrumney
Copy link
Member

As @skuethe says - this can be achieved with #32 so closing this. Thanks for the contribution though, aprreciated

@owenrumney owenrumney closed this Jan 19, 2022
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.

3 participants