Skip to content

Conversation

DmytroLynda
Copy link
Contributor

@DmytroLynda DmytroLynda commented Oct 7, 2025

This PR contains:

  • .cycodeignore support for repository scans
  • .cycodeignore support for commit range scans
  • .cycodeignore allowance (decides if the ignore file can be used for this repository)

What already existed:

  • .cycodeignore support for path scans (by walk_ignore.py)

The get_scan_configuration was extended with a new argument - remote_url. It is used by API to identify the repository and find relevant AllowanceConfiguration which enables/disables .cycodeignore usage in the repository

@gotbadger
Copy link
Collaborator

gotbadger commented Oct 7, 2025

@DmytroLynda please make sure your signing key is correctly setup, its required for all commits.

@DmytroLynda DmytroLynda force-pushed the CM-53929-add-full-support-of-cycodeignore-in-cli branch 4 times, most recently from bcdb6e4 to 9595e33 Compare October 7, 2025 12:41
@DmytroLynda DmytroLynda force-pushed the CM-53929-add-full-support-of-cycodeignore-in-cli branch from 9595e33 to a5f32c2 Compare October 7, 2025 12:43
@DmytroLynda
Copy link
Contributor Author

Hi @gotbadger
I have made all fixes you asked for.

I run manual tests to ensure that it works all together.
Additionally I tested the CLI with/without API changes to ensure backward compatibility

@gotbadger gotbadger merged commit c0bf190 into cycodehq:main Oct 10, 2025
23 checks passed
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.

2 participants