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

✏️ Fix minor typo in fastapi/applications.py #11099

Merged
merged 1 commit into from
Feb 6, 2024

Conversation

JacobHayes
Copy link
Contributor

Fixes a minor typo duplicating an a.

@alejsdev alejsdev changed the title Fix minor FastAPI doc typo ✏️ Fix minor typo in fastapi/applications.py Feb 6, 2024
@alejsdev alejsdev added the docs Documentation about how to use FastAPI label Feb 6, 2024
@alejsdev alejsdev merged commit 0880a5c into fastapi:master Feb 6, 2024
21 checks passed
@alejsdev
Copy link
Member

alejsdev commented Feb 6, 2024

Thanks @JacobHayes 🤓

@JacobHayes JacobHayes deleted the fix-doc branch February 6, 2024 23:30
gisbdzhch pushed a commit to gisktzh/gb3-search_api that referenced this pull request Feb 29, 2024
This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [fastapi](https://github.com/tiangolo/fastapi) | `==0.109.2` -> `==0.110.0` | [![age](https://developer.mend.io/api/mc/badges/age/pypi/fastapi/0.110.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://developer.mend.io/api/mc/badges/adoption/pypi/fastapi/0.110.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://developer.mend.io/api/mc/badges/compatibility/pypi/fastapi/0.109.2/0.110.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/pypi/fastapi/0.109.2/0.110.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) |

---

### Release Notes

<details>
<summary>tiangolo/fastapi (fastapi)</summary>

### [`v0.110.0`](https://github.com/tiangolo/fastapi/releases/tag/0.110.0)

[Compare Source](fastapi/fastapi@0.109.2...0.110.0)

##### Breaking Changes

-   🐛 Fix unhandled growing memory for internal server errors, refactor dependencies with `yield` and `except` to require raising again as in regular Python. PR [#&#8203;11191](fastapi/fastapi#11191) by [@&#8203;tiangolo](https://github.com/tiangolo).
    -   This is a breaking change (and only slightly) if you used dependencies with `yield`, used `except` in those dependencies, and didn't raise again.
    -   This was reported internally by [@&#8203;rushilsrivastava](https://github.com/rushilsrivastava) as a memory leak when the server had unhandled exceptions that would produce internal server errors, the memory allocated before that point would not be released.
    -   Read the new docs: [Dependencies with `yield` and `except`](https://fastapi.tiangolo.com/tutorial/dependencies/dependencies-with-yield/#dependencies-with-yield-and-except).

In short, if you had dependencies that looked like:

```Python
def my_dep():
    try:
        yield
    except SomeException:
        pass
```

Now you need to make sure you raise again after `except`, just as you would in regular Python:

```Python
def my_dep():
    try:
        yield
    except SomeException:
        raise
```

##### Docs

-   ✏️ Fix minor typos in `docs/ko/docs/`. PR [#&#8203;11126](fastapi/fastapi#11126) by [@&#8203;KaniKim](https://github.com/KaniKim).
-   ✏️ Fix minor typo in `fastapi/applications.py`. PR [#&#8203;11099](fastapi/fastapi#11099) by [@&#8203;JacobHayes](https://github.com/JacobHayes).

##### Translations

-   🌐 Add German translation for `docs/de/docs/reference/background.md`. PR [#&#8203;10820](fastapi/fastapi#10820) by [@&#8203;nilslindemann](https://github.com/nilslindemann).
-   🌐 Add German translation for `docs/de/docs/reference/templating.md`. PR [#&#8203;10842](fastapi/fastapi#10842) by [@&#820...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Documentation about how to use FastAPI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants