Skip to content

feat: Lint with ruff #4043

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

Merged
jeremyary merged 10 commits into from
Apr 3, 2024
Merged

feat: Lint with ruff #4043

Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
af97594
feat: Lin with ruff
dmartinol Mar 26, 2024
26db7ca
fea: replace black with ruff format options
dmartinol Mar 26, 2024
8bd6ca6
fea: files reformatted due to ruff deviations from black
dmartinol Mar 26, 2024
7027d50
fix: restored mypy in lint-python target
dmartinol Mar 27, 2024
14525b0
fix: fixed formatting error
dmartinol Mar 27, 2024
a6c70b7
adding ruff format --check
dmartinol Mar 29, 2024
b622a08
Merge remote-tracking branch 'upstream/master' into ruff_linter
dmartinol Apr 2, 2024
b478cc2
Formatting updated files
dmartinol Apr 2, 2024
16c0a9b
Adding section separator
dmartinol Apr 2, 2024
9f4dd9a
rebased to main
dmartinol Apr 3, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fix: restored mypy in lint-python target 
Signed-off-by: Daniele Martinoli <86618610+dmartinol@users.noreply.github.com>
  • Loading branch information
@dmartinol
dmartinol committed Mar 27, 2024
commit 7027d508b00680316327afb1945672e3af40087b
2 changes: 1 addition & 1 deletion Makefile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -321,7 +321,7 @@ format-python:
cd ${ROOT_DIR}/sdk/python; python -m ruff format feast/ tests/

lint-python:
# cd ${ROOT_DIR}/sdk/python; python -m mypy feast
cd ${ROOT_DIR}/sdk/python; python -m mypy feast
cd ${ROOT_DIR}/sdk/python; ruff check feast/ tests/
Comment thread
sudohainguyen marked this conversation as resolved.
Outdated

# Java
Expand Down