Skip to content

Improve missing rules file status and messaging#11

Merged
dkargatzis merged 1 commit intomainfrom
refactor/rules-status-check-and-test-support
Aug 3, 2025
Merged

Improve missing rules file status and messaging#11
dkargatzis merged 1 commit intomainfrom
refactor/rules-status-check-and-test-support

Conversation

@dkargatzis
Copy link
Member

Overview

This PR addresses described in issue #9:

  • Misleading "success" status when rules file is missing - addressed by providing clear, actionable feedback when rules are not configured
  • Support for testing rules in feature branches - still under consideration

Changes

  • Change missing rules file status from "success" to "neutral"
  • Add clear setup instructions with examples
  • Clarify that rules are read from main branch only
  • Update documentation links to correct configuration page

- Change missing rules file status from "success" to "neutral"
- Add clear setup instructions with examples
- Clarify that rules are read from main branch only
- Update documentation links to correct configuration page

Addresses issue #9 (Point #1: misleading success status when rules file is missing)

Signed-off-by: Dimitris Kargatzis <dkargatzis@gmail.com>
@dkargatzis dkargatzis self-assigned this Jul 29, 2025
@dkargatzis dkargatzis marked this pull request as draft July 29, 2025 22:26
@dkargatzis dkargatzis added the enhancement New feature or request label Jul 29, 2025
@watchflow
Copy link

watchflow bot commented Jul 29, 2025

Watchflow rules file not found

The file .watchflow/rules.yaml is missing from your repository.

How to fix:

  1. Create a file at .watchflow/rules.yaml in your repository root.
  2. Add your rules in the following format:
    rules:
      - id: example-rule
        description: Example rule description
        ...
  3. Read the documentation for more details.

After adding the file, push your changes to re-run validation.

@dkargatzis dkargatzis marked this pull request as ready for review August 3, 2025 11:33
@dkargatzis dkargatzis merged commit 6561baa into main Aug 3, 2025
2 checks passed
@dkargatzis dkargatzis deleted the refactor/rules-status-check-and-test-support branch August 3, 2025 11:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant