Skip to content

Conversation

cx-artur-ribeiro
Copy link
Contributor

@cx-artur-ribeiro cx-artur-ribeiro commented Jun 16, 2025

Closes #

Note: The changes on this PR were originally done by a community member. The objective of this PR was to refactor and improve some of the code, as well as to test the implementation since the original PR was done in a private kics fork.
I would like to personally congratulate JonasCorsen for taking the time to improve KICS and for making this effort! 🎆

Reason for Proposed Changes

  • Added support for on Bicep files for kics-scan comments , following the future-improvements notes;

Proposed Changes

  • Removed outdated information from the documentation and added new useful information on bicep comments files;
  • Users can now ignore lines by comments on the original bicep file;
  • Lines containing comments will now also be excluded from the scan;
  • Added a command to generate bicep antlr files to the Makefile;
  • Added Unit Tests to cover the new bicep behaviour;

I submit this contribution under the Apache-2.0 license.

@cx-artur-ribeiro cx-artur-ribeiro self-assigned this Jun 16, 2025
@github-actions github-actions bot added the feature New feature label Jun 16, 2025
Copy link
Contributor

github-actions bot commented Jun 16, 2025

kics-logo

KICS version: v2.1.11

Category Results
CRITICAL CRITICAL 0
HIGH HIGH 0
MEDIUM MEDIUM 0
LOW LOW 0
INFO INFO 0
TRACE TRACE 0
TOTAL TOTAL 0
Metric Values
Files scanned placeholder 1
Files parsed placeholder 1
Files failed to scan placeholder 0
Total executed queries placeholder 47
Queries failed to execute placeholder 0
Execution time placeholder 0

@chielboogaard
Copy link

Hi,

Just curious on the status of this and if I can help in anyway?

@chielboogaard
Copy link

Is there any way I can help with this? I would like to start implementing kics in my workflow.

@cx-artur-ribeiro
Copy link
Contributor Author

Hey @chielboogaard, thanks for checking in, and sorry for the delayed reply!

Right now we’re waiting on QA resources before we can move this feature forward, since a few other priorities are still in progress. Once we’re able to resume, I’ll make sure to update you. The issue and related PR will be closed once the implementation is merged.

I don’t think there’s much to be done here besides the QA process, but if something comes up where help is possible, I’ll definitely reach out.

Really appreciate your patience and your interest in improving KICS!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants