Skip to content

Clarify error message for sync_list rule - #3761

Merged
abraunegg merged 4 commits into
abraunegg:masterfrom
goekce:goekce-patch-1
Jun 28, 2026
Merged

abraunegg merged 4 commits into
abraunegg:masterfrom
goekce:goekce-patch-1

Conversation

@goekce

@goekce goekce commented Jun 28, 2026

Copy link
Copy Markdown
Contributor

Based on #3753 (reply in thread)

Update error message for invalid sync_list rule to clarify that directories at the root level must be manually added each.

Update error message for invalid sync_list rule to clarify that directories at the root level must be manually added each.
Copilot AI review requested due to automatic review settings June 28, 2026 20:35

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the error message shown when an invalid sync_list “include root” rule is detected, clarifying how root-level syncing should be configured.

Changes:

  • Expanded the invalid sync_list root-rule error message to explain that root directories must be included explicitly (wildcard root includes are not allowed).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/clientSideFiltering.d Resolved
@abraunegg
abraunegg merged commit 32a0110 into abraunegg:master Jun 28, 2026
9 checks passed
@abraunegg

Copy link
Copy Markdown
Owner

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

Repository owner locked and limited conversation to collaborators Jul 6, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants