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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

馃帹 Fix format in docs explaining datetime parameter type #220

Merged
merged 2 commits into from May 12, 2022

Conversation

DiegoPiloni
Copy link
Contributor

Fixes a formatting problem mentioned in issue #172 in comment: #172 (comment)

@codecov
Copy link

codecov bot commented Jan 13, 2021

Codecov Report

Merging #220 (9835bd6) into master (c3b1c86) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master      #220   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          252       252           
  Lines         5287      5287           
=========================================
  Hits          5287      5287           

Continue to review full report at Codecov.

Legend - Click here to learn more
螖 = absolute <relative> (impact), 酶 = not affected, ? = missing data
Powered by Codecov. Last update c3b1c86...9835bd6. Read the comment docs.

Copy link
Collaborator

@svlandeg svlandeg left a comment

Choose a reason for hiding this comment

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

Thanks for the fix!

@svlandeg svlandeg added the docs Improvements or additions to documentation label Apr 21, 2022
@tiangolo tiangolo changed the title Add newline before list of formats in DateTime parameter type docs 馃帹 Fix format in docs explaining datetime parameter type May 12, 2022
@tiangolo
Copy link
Owner

Awesome, thank you @DiegoPiloni! 馃嵃

And thanks @svlandeg! 馃 馃崻

@github-actions
Copy link

馃摑 Docs preview for commit 9835bd6 at: https://627d594f42f2ce1527d93d9e--typertiangolo.netlify.app

@tiangolo tiangolo merged commit e61aa15 into tiangolo:master May 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants