-
Notifications
You must be signed in to change notification settings - Fork 594
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
Run aspell on the documentation #407
Conversation
@@ -39,7 +39,7 @@ eagerly consume the entity and make it available in memory: | |||
### Discarding the HTTP Response Entity (Client) | |||
|
|||
Sometimes when calling HTTP services we do not care about their response payload (e.g. all we care about is the response code), | |||
yet as explained above entity still has to be consumed in some way, otherwise we'll be exherting back-pressure on the | |||
yet as explained above entity still has to be consumed in some way, otherwise we'll be exerting back-pressure on the |
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.
Not sure about this one!
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.
Seems correct
Did you use US or UK english? We tend to use US english (though I find that weird being UK language biased ;-)) |
Awesome idea btw |
@@ -39,7 +39,7 @@ eagerly consume the entity and make it available in memory: | |||
### Discarding the HTTP Response Entity (Client) | |||
|
|||
Sometimes when calling HTTP services we do not care about their response payload (e.g. all we care about is the response code), | |||
yet as explained above entity still has to be consumed in some way, otherwise we'll be exherting back-pressure on the | |||
yet as explained above entity still has to be consumed in some way, otherwise we'll be exerting back-pressure on the |
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.
... also found here
I used en_US, but did not convert British to American spelling: specialised => specialized etc. |
If I get time I might try to come up with some scripts to run aspell or hunspell with a pregenerated dictionary to reduce noise from code snippets etc. |
Test FAILed. |
When did you branch off this PR? Can you rebase just to be sure that fix is included here? |
I branched on top of current master
|
Rechecked and it is in fact a new failure for the test case with seven parameters. https://jenkins.akka.io:8498/job/pr-validator-akka-http/160/consoleFull
|
Test PASSed. |
Could the test failure be caused by a race condition in how TestUtil finds available ports and the test binds to it? Let me know if I should create an issue. |
We'll work on fleaky tests separately from this issue, I'll open tickets for each failure. |
LGTM, thanks! |
* Run aspell on the documentation * Spell check README and CONTRIBUTING docs
No description provided.