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.

Sign up for GitHub

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

Jump to bottom

🚀 Add Ruff Linter #1651

Open
wants to merge 179 commits into
base: main
Choose a base branch
from
Open

🚀 Add Ruff Linter #1651

wants to merge 179 commits into from

Conversation

Smartappli
Copy link
Contributor

@Smartappli Smartappli commented Aug 2, 2024 •
edited
Loading

Add Ruff Linter

Smartappli and others added 29 commits August 15, 2024 07:41
@Smartappli
Update settings.py
03f8007
@Smartappli
Update llama_types.py
85b1930
@Smartappli
Update llama_cpp.py
a9a7558
@Smartappli
Update ruff.toml
b58c8b5
@Smartappli
Update llama_types.py
683868b
@Smartappli
Update llama_types.py
a5f16dc
@Smartappli
Update llama_cache.py
a35e3f2
@Smartappli
Update llama_cpp.py
ca3fc20
@Smartappli
Update llama.py
9a13636
@Smartappli
Update app.py
e15563f
@Smartappli
Update types.py
82bead9
@Smartappli
Update errors.py
2d9ee84
@Smartappli
Update cli.py
e133736
@Smartappli
Update model.py
5af3b53
@Smartappli
Update settings.py
4494458
@Smartappli
Update test.yaml
55ba31a
@Smartappli
Update test.yaml
42e15fe
@Smartappli
Update test.yaml
00d9c9c
@Smartappli
Update test.yaml
09d4a2e
@Smartappli
Update test.yaml
8614ee7
@Smartappli
Update test.yaml
022d2db
@Smartappli
Update ruff.toml
8e111ce
@Smartappli
Update ruff.toml
34c2087
@Smartappli
Update .pre-commit-config.yaml
28e4dbd
@Smartappli
Update .pre-commit-config.yaml
04d2868
@Smartappli
Update .pre-commit-config.yaml
1ac586f
@Smartappli
Update .pre-commit-config.yaml
6fd538c
@abetlen
Merge branch 'main' into Linters
d2b26f8
@Smartappli
Update .pre-commit-config.yaml
2dd0fce
@lukestanley
Copy link
Contributor

That's a lot commits you've got there. What's the idea here? @Smartappli

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@Smartappli @gaby @lukestanley @abetlen