Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

(maybe) Fixes a typo in the rspamd docs #682

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

reind33r
Copy link

Currently, to add rspamd, the docs suggest to

add rspamd check with default configuration into appropriate check block (probably in local_routing):

checks {
   ...
   rspamd
}```

Adding this block in local_routing results in a failure to start, because in this context only check is valid.

(I see that checks exist at top-level, but if we mention local_routing and appropriate check block, it makes more sense to me that the code block is also check)

@codecov-commenter
Copy link

codecov-commenter commented Feb 25, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 52.65%. Comparing base (798c411) to head (bb2c0c9).

Additional details and impacted files
@@            Coverage Diff             @@
##              dev     #682      +/-   ##
==========================================
- Coverage   52.68%   52.65%   -0.03%     
==========================================
  Files         129      129              
  Lines       13369    13369              
==========================================
- Hits         7043     7039       -4     
- Misses       5660     5663       +3     
- Partials      666      667       +1     
Flag Coverage Δ
integration 34.36% <ø> (+0.09%) ⬆️
unit 52.16% <ø> (-0.05%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants