Skip to content

Commit

Permalink
Merge pull request #8 from jameslamb/cmake-pin
Browse files Browse the repository at this point in the history
skip CMake 3.30.0, add flake8 configuration
  • Loading branch information
BradReesWork authored Jul 9, 2024
2 parents 5fae539 + 996298f commit 8e6619c
Show file tree
Hide file tree
Showing 4 changed files with 31 additions and 6 deletions.
25 changes: 25 additions & 0 deletions .flake8
Original file line number Diff line number Diff line change
@@ -0,0 +1,25 @@
# Copyright (c) 2024, NVIDIA CORPORATION.

[flake8]
filename = *.py, *.pyx, *.pxd, *.pxi
exclude = __init__.py, *.egg, build, docs, .git
force-check = True
max-line-length = 88
ignore =
# line break before binary operator
W503,
# whitespace before :
E203
per-file-ignores =
# Rules ignored only in Cython:
# E211: whitespace before '(' (used in multi-line imports)
# E225: Missing whitespace around operators (breaks cython casting syntax like <int>)
# E226: Missing whitespace around arithmetic operators (breaks cython pointer syntax like int*)
# E227: Missing whitespace around bitwise or shift operator (Can also break casting syntax)
# E275: Missing whitespace after keyword (Doesn't work with Cython except?)
# E402: invalid syntax (works for Python, not Cython)
# E999: invalid syntax (works for Python, not Cython)
# W504: line break after binary operator (breaks lines that end with a pointer)
*.pyx: E211, E225, E226, E227, E275, E402, E999, W504
*.pxd: E211, E225, E226, E227, E275, E402, E999, W504
*.pxi: E211, E225, E226, E227, E275, E402, E999, W504
2 changes: 1 addition & 1 deletion conda/environments/all_cuda-118_arch-x86_64.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ channels:
- nvidia
dependencies:
- breathe
- cmake>=3.26.4
- cmake>=3.26.4,!=3.30.0
- cuda-nvtx
- cuda-version=11.8
- cudatoolkit
Expand Down
2 changes: 1 addition & 1 deletion conda/environments/all_cuda-122_arch-x86_64.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ channels:
- nvidia
dependencies:
- breathe
- cmake>=3.26.4
- cmake>=3.26.4,!=3.30.0
- cuda-cudart-dev
- cuda-nvtx-dev
- cuda-profiler-api
Expand Down
8 changes: 4 additions & 4 deletions dependencies.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,7 @@ files:
- py_version
- test_python_common
- test_python_pylibcugraph

py_build_cugraph_dgl:
output: pyproject
pyproject_dir: python/cugraph-dgl
Expand Down Expand Up @@ -110,8 +110,8 @@ files:
includes:
- test_python_common
- depends_on_pylibwholegraph


cugraph_dgl_dev:
matrix:
cuda: ["11.8"]
Expand Down Expand Up @@ -199,7 +199,7 @@ dependencies:
common:
- output_types: [conda, pyproject]
packages:
- &cmake_ver cmake>=3.26.4
- &cmake_ver cmake>=3.26.4,!=3.30.0
- ninja

docs:
Expand Down

0 comments on commit 8e6619c

Please sign in to comment.