Skip to content
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

Remove unnecessary word from http_cache.rst #5714

Closed
wants to merge 1 commit into from
Closed

Remove unnecessary word from http_cache.rst #5714

wants to merge 1 commit into from

Conversation

zatikbalazs
Copy link
Contributor

Q A
Doc fix? yes
New docs? no
Applies to 2.3, 2.4, 2.5, 2.6, 2.7
Fixed tickets -

@javiereguiluz
Copy link
Member

👍 another easy win! By the way, let's ask @xabbuh if we could also do another change in this PR: the comma before the and word could probably be removed.

@xabbuh
Copy link
Member

xabbuh commented Sep 23, 2015

👍 and I also agree with removing the comma.

xabbuh added a commit that referenced this pull request Sep 23, 2015
This PR was submitted for the 2.7 branch but it was merged into the 2.3 branch instead (closes #5714).

Discussion
----------

Remove unnecessary word from http_cache.rst

| Q             | A
| ------------- | ---
| Doc fix?      | yes
| New docs?     | no
| Applies to    | 2.3, 2.4, 2.5, 2.6, 2.7
| Fixed tickets | -

Commits
-------

e0b32e5 Remove unnecessary word from http_cache.rst
@xabbuh
Copy link
Member

xabbuh commented Sep 23, 2015

Thank you @zatikbalazs. I merged your changes into the 2.3 branch (that's why it is shown as closed) and merged the comma in 290997c.

@xabbuh xabbuh closed this Sep 23, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants