FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

🔧 Add pre-commit config and first run by tiangolo · Pull Request #416 · fastapi/sqlmodel · GitHub

Repository navigation

🔧 Add pre-commit config and first run - #416

Closed
tiangolo wants to merge 7 commits into
mainfrom
pre-commit
Closed

tiangolo wants to merge 7 commits into
mainfrom
pre-commit

Conversation

Copy link
Copy Markdown
Member

🔧 Add pre-commit config and first run

Copy link
Copy Markdown
Contributor

codecov Bot commented Aug 27, 2022 •
edited
Loading

Copy link
Copy Markdown

Codecov Report

Merging #416 (244f477) into main (36b0c1b) will decrease coverage by 0.14%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #416      +/-   ##
==========================================
- Coverage   97.49%   97.35%   -0.15%     
==========================================
  Files         181      181              
  Lines        6038     6038              
==========================================
- Hits         5887     5878       -9     
- Misses        151      160       +9     
Impacted Files Coverage Δ
sqlmodel/sql/expression.py 74.35% <0.00%> (-23.08%) ⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Copy link
Copy Markdown
Contributor

📝 Docs preview for commit 244f477 at: https://630a718a0a0b95381d267c53--sqlmodel.netlify.app

tiangolo added the internal Internal changes label Oct 22, 2023

Copy link
Copy Markdown
Member Author

This was done in #661 and #667

tiangolo closed this Oct 23, 2023
tiangolo deleted the pre-commit branch October 23, 2023 08:20
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

internal Internal changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant


Back | FazBrowse Home | New Git URL