Merge pull request Point72#223 from Point72/tkp/lint

Pin linters to avoid noise, run auto fixers after latest changes and pinning
Carreau · May 13, 2024 · 12cceeb · 12cceeb
2 parents 78e7aca + 471e142
commit 12cceeb
Show file tree

Hide file tree

Showing 5 changed files with 18 additions and 18 deletions.
diff --git a/conda/dev-environment-unix.yml b/conda/dev-environment-unix.yml
@@ -8,7 +8,7 @@ dependencies:
  - build
  - bump2version>=1
  - cmake
- - codespell
+ - codespell>=2.2.6,<2.3
  - compilers
  - cyrus-sasl
  - exprtk
@@ -23,7 +23,7 @@ dependencies:
  - libboost-headers
  - lz4-c
  - mamba
- - mdformat
+ - mdformat>=0.7.17,<0.8
  - ninja
  - numpy
  - pillow
@@ -43,7 +43,7 @@ dependencies:
  - rapidjson
  - requests
  - ruamel.yaml
- - ruff
+ - ruff>=0.3,<0.4
  - scikit-build
  - slack-sdk
  - sqlalchemy

diff --git a/docs/wiki/concepts/Execution-Modes.md b/docs/wiki/concepts/Execution-Modes.md
@@ -41,14 +41,14 @@ As always, `csp.now()` should still be used in `csp.node` code, even when runnin
 
 When consuming data from input adapters there are three choices on how one can consume the data:
 
-| PushMode       | EngineMode | Description |
-| :-------       | :--------- | :---------- |
-| **LAST_VALUE** | Simulation | all ticks from input source with duplicate timestamps (on the same timeseries) will tick once with the last value on a given timestamp |
-|           | Realtime   | all ticks that occurred since previous engine cycle will collapse / conflate to the latest value |
+| PushMode           | EngineMode | Description                                                                                                                                                       |
+| :----------------- | :--------- | :---------------------------------------------------------------------------------------------------------------------------------------------------------------- |
+| **LAST_VALUE**     | Simulation | all ticks from input source with duplicate timestamps (on the same timeseries) will tick once with the last value on a given timestamp                            |
+|                    | Realtime   | all ticks that occurred since previous engine cycle will collapse / conflate to the latest value                                                                  |
 | **NON_COLLAPSING** | Simulation | all ticks from input source with duplicate timestamps (on the same timeseries) will tick once per engine cycle. subsequent cycles will execute with the same time |
-|           | Realtime | all ticks that occurred since previous engine cycle will be ticked across subsequent engine cycles as fast as possible |
-| **BURST**      | Simulation | all ticks from input source with duplicate timestamps (on the same timeseries) will tick once with a list of all values |
-|           | Realtime | all ticks that occurred since previous engine cycle will tick once with a list of all the values |
+|                    | Realtime   | all ticks that occurred since previous engine cycle will be ticked across subsequent engine cycles as fast as possible                                            |
+| **BURST**          | Simulation | all ticks from input source with duplicate timestamps (on the same timeseries) will tick once with a list of all values                                           |
+|                    | Realtime   | all ticks that occurred since previous engine cycle will tick once with a list of all the values                                                                  |
 
 ## Realtime Group Event Synchronization
 

diff --git a/docs/wiki/dev-guides/Build-CSP-from-Source.md b/docs/wiki/dev-guides/Build-CSP-from-Source.md
@@ -198,11 +198,11 @@ By default, we pull and build dependencies with [vcpkg](https://vcpkg.io/en/). W
 
 CSP has listing and auto formatting.
 
-| Language | Linter | Autoformatter | Description |
-| :------- | :----- | :------------ | :---------- |
-| C++      | `clang-format` | `clang-format` | Style |
-| Python   | `ruff`         | `ruff` | Style |
-| Python   | `isort`         | `isort` | Imports |
+| Language | Linter         | Autoformatter  | Description |
+| :------- | :------------- | :------------- | :---------- |
+| C++      | `clang-format` | `clang-format` | Style       |
+| Python   | `ruff`         | `ruff`         | Style       |
+| Python   | `isort`        | `isort`        | Imports     |
 
 **C++ Linting**
 

diff --git a/pyproject.toml b/pyproject.toml
@@ -60,9 +60,9 @@ develop = [
     "twine",
     "wheel",
     # lint
-    "codespell",
+    "codespell>=2.2.6,<2.3",
     "isort>=5,<6",
-    "mdformat",
+    "mdformat>=0.7.17,<0.8",
     "ruff>=0.3,<0.4",
     # test
     "pytest",

diff --git a/setup.py b/setup.py
@@ -64,7 +64,7 @@
     )
 
     if VCPKG_TRIPLET is not None:
-        cmake_args.append( f"-DVCPKG_TARGET_TRIPLET={VCPKG_TRIPLET}" )
+        cmake_args.append(f"-DVCPKG_TARGET_TRIPLET={VCPKG_TRIPLET}")
 else:
     cmake_args.append("-DCSP_USE_VCPKG=OFF")