Skip to content

fix(bicep): support multi-line function calls#7457

Open
safejulian wants to merge 3 commits intobridgecrewio:mainfrom
safejulian:bicep_function_fix
Open

fix(bicep): support multi-line function calls#7457
safejulian wants to merge 3 commits intobridgecrewio:mainfrom
safejulian:bicep_function_fix

Conversation

@safejulian
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.

Description

This PR fixes issue 7316; where a request was made for support of multiline function calls. This is an issue for an organisation that I know.

The PR has a test to demonstrate the issue, and updates the pycep-parser library version to improve Bicep parsing. The test now passes (as do all others)

Tested on some real world Bicep code that has multi-line functions, and my local Checkov reported fewer parsing errors.

pycep-parsee 0.7.0 doesn't support Python 3.9 - presumably as 3.9 is unsupported now. So I had to bump the Python version.

Fixes #7316

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

(no runtime code changed, no documentation needed for minor enhancement)

This commit updates the Pycep library to improve Bicep parsing, with
tests to demonstrate that parsing is successful for the issue raised in
issue 7316.

Tested on some real world Bicep code that has multi-line functions, and
Checkov reports fewer parsing errors.
@OfekShimko
Copy link
Copy Markdown
Contributor

Hi safejulian,

Thank you for your contribution! We are currently reviewing your changes, specifically as they relate to upgrading Checkov to support Python 3.10 and above.

We appreciate your patience and your help in improving the project!

@safejulian
Copy link
Copy Markdown
Author

safejulian commented Mar 12, 2026

Hi safejulian,

Thank you for your contribution! We are currently reviewing your changes, specifically as they relate to upgrading Checkov to support Python 3.10 and above.

Hey @OfekShimko

Thanks for letting me know. I understand the need for extra diligence on the runtime changes.

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.

Support bicep multi-line function calls

2 participants