Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

✏️ Fix typo infastapi/params.py#12143

Merged
alejsdev merged 2 commits intofastapi:masterfrom
surreal30:fix/deprecated_typo
Sep 6, 2024
Merged

✏️ Fix typo infastapi/params.py#12143
alejsdev merged 2 commits intofastapi:masterfrom
surreal30:fix/deprecated_typo

Conversation

@surreal30
Copy link
Contributor

#What

  • There was a spelling mistake.deprecated was spelt wrong

svlandeg reacted with thumbs up emoji
@alejsdevalejsdev changed the titlefix: deprecated spelling✏️ Fix typo infastapi/params.pySep 6, 2024
Copy link
Member

@svlandegsvlandeg left a comment•
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

Good catch, and thanks for the fix! 🙏

Our automated tests are currently failing due to some other reason, we're looking into it. Once that's fixed, we'll be able to merge your PR 🎉

[Update: the tests were fixed and this is now green 🎉]

alejsdev and surreal30 reacted with rocket emoji
@surreal30
Copy link
ContributorAuthor

Thanks. That would be great!

@alejsdevalejsdev merged commitb60d36e intofastapi:masterSep 6, 2024
@alejsdev
Copy link
Member

Good catch, thanks!@surreal30 🤓

@surreal30
Copy link
ContributorAuthor

Thanks ^_^

jvllmr pushed a commit to jvllmr/fastapi that referenced this pull requestSep 6, 2024
black-redoc pushed a commit to black-redoc/fastapi that referenced this pull requestSep 12, 2024
black-redoc pushed a commit to black-redoc/fastapi that referenced this pull requestSep 12, 2024
black-redoc pushed a commit to black-redoc/fastapi that referenced this pull requestSep 12, 2024
s-rigaud pushed a commit to s-rigaud/fastapi that referenced this pull requestJan 23, 2025
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@svlandegsvlandegsvlandeg approved these changes

+1 more reviewer

@JavierSanchezCastroJavierSanchezCastroJavierSanchezCastro approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@surreal30@alejsdev@svlandeg@JavierSanchezCastro

Comments

[8]ページ先頭
©2009-2026 Movatter.jp