Skip to content

feat(docker): disable pattern check#7555

Open
badamowicz wants to merge 1 commit into
bridgecrewio:mainfrom
badamowicz:feature/disable-dockerfile-pattern
Open

feat(docker): disable pattern check#7555
badamowicz wants to merge 1 commit into
bridgecrewio:mainfrom
badamowicz:feature/disable-dockerfile-pattern

Conversation

@badamowicz

@badamowicz badamowicz commented May 26, 2026

Copy link
Copy Markdown

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

# Title: feat(docker): disable pattern check for Dockerfiles
#  Allowed prefixs: dockerfile

Description

This PR addresses the issue mentioned in: #7541 (comment).
It's about having the possibility to disable the patterncheck for Dockerfiles. Therefore I added a new command line argument --ignore-dockerfile-pattern which only applies if also --file is given and the framework is dockerfile.
I'm not very happy with my solution because I am using an environment variable to disable the pattern. The reason I did it this way was that I found no better way to get the command line argument into the runner.
So if anybody can point me to a better way, this would be appreciated.

Checklist:

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have added tests that prove my feature, policy, or fix is effective and works
  • New and existing tests pass locally with my changes

@badamowicz badamowicz marked this pull request as draft May 26, 2026 13:16
@badamowicz badamowicz changed the title [WIP] feat(docker): disable pattern check feat(docker): disable pattern check Jun 5, 2026
@badamowicz badamowicz marked this pull request as ready for review June 5, 2026 06:56
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.

1 participant