Skip to content

Conversation

@JonZeolla
Copy link
Member

Contributor Comments

This updates CI so we scan the CI logs for warnings, errors, deprecation notices.

Right now deprecated items can easily be missed. This at least semi-highlights them, however it is prone to some false positives, for instance it would match on a log that says:

[test] Configuration is valid - found 0 fatal errors, 0 skippable error(s), and 388 rule(s).

Pull Request Checklist

Thank you for submitting a contribution!

Please address the following items:

  • If you are adding a dependency, please explain how it was chosen.
  • If manual testing is needed in order to validate the changes, provide a testing plan and the expected results.
  • Validate that documentation is accurate and aligned to any project updates or additions.

Copy link

@ai-coding-guardrails ai-coding-guardrails bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've got 4 comments for you to consider

Reviewed with 🤟 by Zenable

@JonZeolla JonZeolla enabled auto-merge (squash) July 16, 2025 17:47
@JonZeolla JonZeolla merged commit a84ef6d into main Jul 16, 2025
7 checks passed
@JonZeolla JonZeolla deleted the add-deprecated-monitor branch July 16, 2025 18:03
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