Skip to content

Commit 69f27ad

Browse files
authored
Merge pull request #712 from nipype/pre-commit-ci-update-config
pre-commit updates and code fixes
2 parents 95f94d5 + 6cbeace commit 69f27ad

21 files changed

+36
-17
lines changed

.pre-commit-config.yaml

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2,23 +2,23 @@
22
# See https://pre-commit.com/hooks.html for more hooks
33
repos:
44
- repo: https://github.com/pre-commit/pre-commit-hooks
5-
rev: v4.4.0
5+
rev: v4.5.0
66
hooks:
77
- id: trailing-whitespace
88
- id: end-of-file-fixer
99
- id: check-yaml
1010
- id: check-added-large-files
1111
- repo: https://github.com/psf/black
12-
rev: 23.9.1
12+
rev: 24.1.1
1313
hooks:
1414
- id: black
1515
- repo: https://github.com/codespell-project/codespell
16-
rev: v2.2.5
16+
rev: v2.2.6
1717
hooks:
1818
- id: codespell
1919
additional_dependencies:
2020
- tomli
2121
- repo: https://github.com/PyCQA/flake8
22-
rev: 6.1.0
22+
rev: 7.0.0
2323
hooks:
2424
- id: flake8

docs/changes.rst

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ Release Notes
99
* adding simple validators to input spec (using ``attr.validator``)
1010
* adding ``create_dotfile`` for workflows, that creates graphs as dotfiles (can convert to other formats if dot available)
1111
* adding a simple user guide with ``input_spec`` description
12-
* expanding docstrings for ``State``, ``audit`` and ``messanger``
12+
* expanding docstrings for ``State``, ``audit`` and ``messenger``
1313
* updating syntax to newer python
1414

1515
0.7.0

docs/sphinxext/github_link.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22
This script comes from scikit-learn:
33
https://github.com/scikit-learn/scikit-learn/blob/master/doc/sphinxext/github_link.py
44
"""
5+
56
import inspect
67
import os
78
import subprocess

pydra/__init__.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@
55
first-class operations. It forms the core of the Nipype 2.0 ecosystem.
66
77
"""
8+
89
# This call enables pydra.tasks to be used as a namespace package when installed
910
# in editable mode. In normal installations it has no effect.
1011
__path__ = __import__("pkgutil").extend_path(__path__, __name__)

pydra/engine/__init__.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
"""The core of the workflow engine."""
2+
23
from .submitter import Submitter
34
from .core import Workflow
45
from .task import AuditFlag, ShellCommandTask

pydra/engine/audit.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
"""Module to keep track of provenance information."""
2+
23
import os
34
import json
45
import attr

pydra/engine/core.py

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
"""Basic processing graph elements."""
2+
23
import abc
34
import json
45
import logging
@@ -59,7 +60,7 @@ class TaskBase:
5960
"""
6061
A base structure for the nodes in the processing graph.
6162
62-
Tasks are a generic compute step from which both elemntary tasks and
63+
Tasks are a generic compute step from which both elementary tasks and
6364
:class:`Workflow` instances inherit.
6465
6566
"""
@@ -545,7 +546,7 @@ def _run(self, rerun=False, environment=None, **kwargs):
545546
self.hooks.post_run_task(self, result)
546547
self.audit.finalize_audit(result)
547548
save(output_dir, result=result, task=self)
548-
# removing the additional file with the chcksum
549+
# removing the additional file with the checksum
549550
(self.cache_dir / f"{self.uid}_info.json").unlink()
550551
# # function etc. shouldn't change anyway, so removing
551552
orig_inputs = {
@@ -1251,7 +1252,7 @@ async def _run(self, submitter=None, rerun=False, **kwargs):
12511252
self.hooks.post_run_task(self, result)
12521253
self.audit.finalize_audit(result=result)
12531254
save(output_dir, result=result, task=self)
1254-
# removing the additional file with the chcksum
1255+
# removing the additional file with the checksum
12551256
(self.cache_dir / f"{self.uid}_info.json").unlink()
12561257
os.chdir(cwd)
12571258
self.hooks.post_run(self, result)

pydra/engine/environments.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
class Environment:
77
"""
88
Base class for environments that are used to execute tasks.
9-
Right now it is asssumed that the environment, including container images,
9+
Right now it is assumed that the environment, including container images,
1010
are available and are not removed at the end
1111
TODO: add setup and teardown methods
1212
"""

pydra/engine/graph.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
"""Data structure to support :class:`~pydra.engine.core.Workflow` tasks."""
2+
23
from copy import copy
34
from pathlib import Path
45
import subprocess as sp
@@ -498,7 +499,7 @@ def _create_dotfile_single_graph(self, nodes, edges):
498499
return dotstr
499500

500501
def export_graph(self, dotfile, ext="png"):
501-
"""exporting dotfile to other format, equires the dot command"""
502+
"""exporting dotfile to other formats requires the dot command"""
502503
available_ext = [
503504
"bmp",
504505
"canon",

pydra/engine/helpers.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
"""Administrative support for the engine framework."""
2+
23
import asyncio
34
import asyncio.subprocess as asp
45
from pathlib import Path

0 commit comments

Comments
 (0)