This is an automated email from the ASF dual-hosted git repository.
fokko pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/iceberg-python.git
The following commit(s) were added to refs/heads/main by this push:
new 451cdba5 Move Ruff configuration to separate config file (#619)
451cdba5 is described below
commit 451cdba568b1e2ce4cc0af1aae577b24755cc9eb
Author: Fokko Driesprong <[email protected]>
AuthorDate: Thu Apr 18 12:32:37 2024 +0200
Move Ruff configuration to separate config file (#619)
Dependabot stopped working after merging:
https://github.com/apache/iceberg-python/commit/b829737940c5cf35d169a784ef45a6bddbc4a645
This broke dependabot since it is now encountering
issues:
https://github.com/apache/iceberg-python/security/dependabot/16
I think it is good to just move the Ruff config to a separate file.
---
pyproject.toml | 64 --------------------------------------------
ruff.toml | 83 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
2 files changed, 83 insertions(+), 64 deletions(-)
diff --git a/pyproject.toml b/pyproject.toml
index 7e5dbc8a..5c907d0e 100644
--- a/pyproject.toml
+++ b/pyproject.toml
@@ -779,67 +779,3 @@ ignore_missing_imports = true
[tool.coverage.run]
source = ['pyiceberg/']
-
-[tool.ruff]
-src = ['pyiceberg','tests']
-extend-exclude = ["dev/provision.py"]
-lint.select = [
- "E", # pycodestyle
- "W", # pycodestyle
- "F", # Pyflakes
- "B", # flake8-bugbear
- "PIE", # flake8-pie
- "C4", # flake8-comprehensions
- "I", # isort
- "UP", # pyupgrade
-]
-lint.ignore = ["E501","E203","B024","B028","UP037"]
-
-# Allow autofix for all enabled rules (when `--fix`) is provided.
-lint.fixable = ["ALL"]
-lint.unfixable = []
-
-# Exclude a variety of commonly ignored directories.
-exclude = [
- ".bzr",
- ".direnv",
- ".eggs",
- ".git",
- ".git-rewrite",
- ".hg",
- ".mypy_cache",
- ".nox",
- ".pants.d",
- ".pytype",
- ".ruff_cache",
- ".svn",
- ".tox",
- ".venv",
- "__pypackages__",
- "_build",
- "buck-out",
- "build",
- "dist",
- "node_modules",
- "venv",
-]
-lint.per-file-ignores = {}
-# Ignore _all_ violations.
-# Same as Black.
-line-length = 130
-
-# Allow unused variables when underscore-prefixed.
-lint.dummy-variable-rgx = "^(_+|(_+[a-zA-Z0-9_]*[a-zA-Z0-9]+?))$"
-
-[tool.ruff.lint.pyupgrade]
-# Preserve types, even if a file imports `from __future__ import annotations`.
-keep-runtime-typing = true
-
-[tool.ruff.lint.isort]
-detect-same-package = true
-lines-between-types = 0
-known-first-party = ["pyiceberg", "tests"]
-section-order = ["future", "standard-library", "third-party", "first-party",
"local-folder"]
-
-[tool.ruff.format]
-quote-style = "preserve"
diff --git a/ruff.toml b/ruff.toml
new file mode 100644
index 00000000..92fb9a9c
--- /dev/null
+++ b/ruff.toml
@@ -0,0 +1,83 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements. See the NOTICE file
+# distributed with this work for additional information
+# regarding copyright ownership. The ASF licenses this file
+# to you under the Apache License, Version 2.0 (the
+# "License"); you may not use this file except in compliance
+# with the License. You may obtain a copy of the License at
+#
+# http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing,
+# software distributed under the License is distributed on an
+# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+# KIND, either express or implied. See the License for the
+# specific language governing permissions and limitations
+# under the License.
+
+src = ['pyiceberg','tests']
+extend-exclude = ["dev/provision.py"]
+
+# Exclude a variety of commonly ignored directories.
+exclude = [
+ ".bzr",
+ ".direnv",
+ ".eggs",
+ ".git",
+ ".git-rewrite",
+ ".hg",
+ ".mypy_cache",
+ ".nox",
+ ".pants.d",
+ ".pytype",
+ ".ruff_cache",
+ ".svn",
+ ".tox",
+ ".venv",
+ "__pypackages__",
+ "_build",
+ "buck-out",
+ "build",
+ "dist",
+ "node_modules",
+ "venv",
+]
+
+# Ignore _all_ violations.
+# Same as Black.
+line-length = 130
+
+[lint]
+select = [
+ "E", # pycodestyle
+ "W", # pycodestyle
+ "F", # Pyflakes
+ "B", # flake8-bugbear
+ "PIE", # flake8-pie
+ "C4", # flake8-comprehensions
+ "I", # isort
+ "UP", # pyupgrade
+]
+ignore = ["E501","E203","B024","B028","UP037"]
+
+# Allow autofix for all enabled rules (when `--fix`) is provided.
+fixable = ["ALL"]
+unfixable = []
+
+per-file-ignores = {}
+
+# Allow unused variables when underscore-prefixed.
+dummy-variable-rgx = "^(_+|(_+[a-zA-Z0-9_]*[a-zA-Z0-9]+?))$"
+
+[lint.pyupgrade]
+# Preserve types, even if a file imports `from __future__ import annotations`.
+keep-runtime-typing = true
+
+[lint.isort]
+detect-same-package = true
+lines-between-types = 0
+known-first-party = ["pyiceberg", "tests"]
+section-order = ["future", "standard-library", "third-party", "first-party",
"local-folder"]
+
+[format]
+quote-style = "preserve"
