Skip to content

Conversation

@shawn-hurley
Copy link
Contributor

@shawn-hurley shawn-hurley commented Feb 26, 2025

  • Some rules had the from as a capability arg, which would be ignored
  • Some rules had a from and were not using the filepaths to filter

fixes #257

@jmle jmle self-requested a review February 26, 2025 23:55
Copy link
Contributor

@jmle jmle left a comment

Choose a reason for hiding this comment

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

Looks good! Good catch

@aufi
Copy link
Member

aufi commented Feb 27, 2025

Going to close and reopen to refresh CI workflows.

@aufi aufi closed this Feb 27, 2025
@aufi aufi reopened this Feb 27, 2025
@github-actions
Copy link

github-actions bot commented Nov 5, 2025

This pull request has been automatically marked as stale because it has not had any activity for 60 days.
It will remain open for visibility and reporting purposes.
Please comment if this PR is still relevant.

@github-actions github-actions bot added the stale label Nov 5, 2025
@shawn-hurley shawn-hurley force-pushed the bugfix/use-filepaths-to-filter branch from 84e6114 to 803d9b2 Compare January 6, 2026 16:56
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 6, 2026

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters (22)
  • default/generated/azure/08-eap-to-azure-appservice-datasource-driver.windup.yaml is excluded by !**/generated/**
  • default/generated/camel3/30-component-changes.groovy.windup.yaml is excluded by !**/generated/**
  • default/generated/camel3/33-java-generic-information.groovy.windup.yaml is excluded by !**/generated/**
  • default/generated/camel3/34-java-generic-information.windup.yaml is excluded by !**/generated/**
  • default/generated/eap6/82-xml-webservices.windup.yaml is excluded by !**/generated/**
  • default/generated/eap6/83-eap4-xml-config.windup.yaml is excluded by !**/generated/**
  • default/generated/eap6/85-jboss-eap5-xml.windup.yaml is excluded by !**/generated/**
  • default/generated/eap7/105-jboss-eap5and6to7-xml.windup.yaml is excluded by !**/generated/**
  • default/generated/eap7/107-eap6-xml.windup.yaml is excluded by !**/generated/**
  • default/generated/eap7/109-hibernate4-xml.windup.yaml is excluded by !**/generated/**
  • default/generated/eap7/110-hibernate4.windup.yaml is excluded by !**/generated/**
  • default/generated/eap7/112-jax-ws.windup.yaml is excluded by !**/generated/**
  • default/generated/eap7/116-hibernate51-53.windup.yaml is excluded by !**/generated/**
  • default/generated/eap7/124-singleton-sessionbean.rhamt.yaml is excluded by !**/generated/**
  • default/generated/eap7/136-weblogic-xml-descriptors.windup.yaml is excluded by !**/generated/**
  • default/generated/eap7/144-websphere-xml.windup.yaml is excluded by !**/generated/**
  • default/generated/eap8/154-hibernate6.windup.yaml is excluded by !**/generated/**
  • default/generated/eap8/161-javaee-to-jakarta-namespaces.windup.yaml is excluded by !**/generated/**
  • default/generated/hibernate/187-hibernate-xml.windup.yaml is excluded by !**/generated/**
  • default/generated/quarkus/201-persistence-to-quarkus.windup.yaml is excluded by !**/generated/**
  • default/generated/technology-usage/08-security-technology-usage.windup.yaml is excluded by !**/generated/**
  • default/generated/technology-usage/22-javaee-technology-usage.windup.yaml is excluded by !**/generated/**

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@shawn-hurley shawn-hurley removed the stale label Jan 6, 2026
@shawn-hurley
Copy link
Contributor Author

@jmle @aufi @eemcmullan Can y'all give another round of reviews after the rebase?

@shawn-hurley shawn-hurley force-pushed the bugfix/use-filepaths-to-filter branch from 803d9b2 to dcf877c Compare January 6, 2026 17:28
* Some rules had the from as a capability arg, which would be ignored
* Some rules had a from and were not using the filepaths to filter

Signed-off-by: Shawn Hurley <shawn@hurley.page>
@shawn-hurley shawn-hurley force-pushed the bugfix/use-filepaths-to-filter branch from dcf877c to de7b0b3 Compare January 8, 2026 17:08
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.

The incident resolved by the LLM is different from the one that the user selects

5 participants