checkov icon indicating copy to clipboard operation
checkov copied to clipboard

fix(general): Fix accidentally merged categories in prompt.py

Open InputUsername opened this issue 1 year ago • 2 comments

User description

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

Description

The missing comma caused backup_and_recovery and convention to be merged into one category preventing either category from being specified.

Checklist:

  • [x] My code follows the style guidelines of this project
  • [x] 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
  • [ ] Any dependent changes have been merged and published in downstream modules

Generated description

Dear maintainer, below is a concise technical summary of the changes proposed in this PR:

Fix a missing comma in prompt.py that caused backup_and_recovery and convention categories to merge, preventing their individual specification.

TopicDetails
Category Fix Correct a syntax error in the CATEGORIES list of Prompt class to ensure proper category separation.
Modified files (1)
  • checkov/common/util/prompt.py
Latest Contributors(2)
EmailCommitDate
[email protected]Improve-type-hints-in-...May 08, 2022
17982571+jmeredith16@u...Check-Generator-CLI-Pr...November 08, 2021
This pull request is reviewed by Baz. Join @InputUsername and the rest of your team on (Baz).

InputUsername avatar Jun 14 '24 07:06 InputUsername

Hi @InputUsername . can you please make sure your source branch is up-to-date with bridgecrewio:main?

itai1357 avatar Jul 18 '24 11:07 itai1357

@itai1357 is this OK or do you want me to rebase?

InputUsername avatar Jul 18 '24 15:07 InputUsername

Hi @InputUsername just pull from main branch and push back.

AdamDev avatar Jan 05 '25 12:01 AdamDev

@AdamDev done!

InputUsername avatar Jan 05 '25 13:01 InputUsername

Thanks for contributing to Checkov! We've automatically marked this issue as stale to keep our issues list tidy, because it has not had any activity for 6 months. It will be closed in 14 days if no further activity occurs. Commenting on this issue will remove the stale tag. If you want to talk through the issue or help us understand the priority and context, feel free to add a comment or join us in the Checkov slack channel at codifiedsecurity.slack.com Thanks!

stale[bot] avatar Jul 06 '25 14:07 stale[bot]

Closing issue due to inactivity. If you feel this is in error, please re-open, or reach out to the community via slack: codifiedsecurity.slack.com Thanks!

stale[bot] avatar Jul 20 '25 20:07 stale[bot]