Skip to content

Commit

Permalink
MAINT: Ignore, CIs, pre-commit hook
Browse files Browse the repository at this point in the history
  • Loading branch information
larsoner committed Dec 13, 2022
1 parent e116472 commit 3f194e5
Show file tree
Hide file tree
Showing 8 changed files with 82 additions and 98 deletions.
141 changes: 50 additions & 91 deletions .circleci/config.yml

Large diffs are not rendered by default.

2 changes: 2 additions & 0 deletions .git-blame-ignore-revs
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
# black
7d03ec2f7bb5d12a747366cecd7e56b306a35d7f
1 change: 1 addition & 0 deletions .github/workflows/run-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,7 @@ jobs:
run: pip install ruff codespell tomli
- run: make ruff
- run: make codespell-error
- uses: psf/black@stable
check-doc:
name: Doc consistency
runs-on: ubuntu-latest
Expand Down
22 changes: 22 additions & 0 deletions .pre-commit-config.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,22 @@
---
# Eventually we should use yamllint, too
files: ^(.*\.(py|yaml))$
exclude: ^(\.[^/]*cache/.*)$
repos:
- repo: https://github.com/psf/black
rev: 22.10.0
hooks:
- id: black
args:
- --safe
- --quiet
- repo: https://github.com/charliermarsh/ruff-pre-commit
rev: v0.0.178
hooks:
- id: ruff
- repo: https://github.com/codespell-project/codespell
rev: v2.2.2
hooks:
- id: codespell
additional_dependencies:
- tomli
2 changes: 0 additions & 2 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -33,8 +33,6 @@ check:
trailing-spaces:
find . -name "*.py" | xargs perl -pi -e 's/[ \t]*$$//'

flake: ruff

ruff:
ruff .
@echo "ruff passed"
Expand Down
8 changes: 5 additions & 3 deletions docs/source/v1.1.md.inc
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,10 @@

[//]: # (### Behavior changes)

[//]: # (### Code health)
### Code health

[//]: # (### Bug fixes)
- Code quality now checked by [black](https://black.readthedocs.io/en/stable/)
and [ruff]()
({{ gh(688) }} by {{ authors.larsoner }} )

- None yet
[//]: # (### Bug fixes)
2 changes: 1 addition & 1 deletion mne_bids_pipeline/_config_import.py
Original file line number Diff line number Diff line change
Expand Up @@ -145,7 +145,7 @@ def _update_with_user_config(
if not name.startswith("__"):
val = getattr(overrides, name)
if log:
msg = f"Overridding config.{name} = {repr(val)}"
msg = f"Overriding config.{name} = {repr(val)}"
logger.info(
**gen_log_kwargs(message=msg, step="", emoji="override", box="╶╴")
)
Expand Down
2 changes: 1 addition & 1 deletion mne_bids_pipeline/tests/datasets.py
Original file line number Diff line number Diff line change
Expand Up @@ -100,7 +100,7 @@ class DATASET_OPTIONS_T(TypedDict):
"openneuro": "ds003775",
"osf": "",
"web": "",
"include": ["sub-01"],
"include": ["sub-010"],
"exclude": [],
},
"ds001810": {
Expand Down

0 comments on commit 3f194e5

Please sign in to comment.