-
Notifications
You must be signed in to change notification settings - Fork 96
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Small docs change #384
base: master
Are you sure you want to change the base?
Small docs change #384
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A thot about that ... 🙃
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would split the piece of text explaining that we're replacement for smtpd
into a separate paragraph, optionally with some kind of highlight / callout.
I enabled building PR previews @ RTD but it fails currently because the RTD config file needs to be updated: https://readthedocs.org/projects/aiosmtpd/builds/23289246/. The GHA linting is failing is unrelated either, because of the unpinned flake8 plugin version, I suppose. |
Filed an issue for this with pointers: #393. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #384 +/- ##
==========================================
- Coverage 98.14% 98.12% -0.02%
==========================================
Files 22 22
Lines 5646 5646
Branches 755 755
==========================================
- Hits 5541 5540 -1
- Misses 68 70 +2
+ Partials 37 36 -1 ☔ View full report in Codecov by Sentry. |
@waynew needs conflict resolution |
See #320 - I did a quick audit and changed the couple of things that could use it. I did not confirm whether or not any of the code samples in the docs have any changes that are inconsistent with the current codebase... My hope is that they were already update and tested 😅
Anyway... this may or may not completely resolve #320.