Skip to content

Commit

Permalink
Merge pull request #2570 from effigies/ci/style
Browse files Browse the repository at this point in the history
CI: Add style checks (missing since Travis got throttled)
  • Loading branch information
effigies authored Oct 5, 2021
2 parents 5fe2c33 + 626aede commit 442e21e
Show file tree
Hide file tree
Showing 3 changed files with 43 additions and 19 deletions.
41 changes: 41 additions & 0 deletions .github/workflows/contrib.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,41 @@
name: Contribution checks

on:
push:
branches:
- master
- maint/*
pull_request:
branches:
- master
- maint/*

defaults:
run:
shell: bash

jobs:
stable:
runs-on: ${{ matrix.os }}
strategy:
matrix:
os: ['ubuntu-latest']
python-version: [3.9]

steps:
- uses: actions/checkout@v2
with:
submodules: recursive
fetch-depth: 0
- name: Set up Python ${{ matrix.python-version }}
uses: actions/setup-python@v2
with:
python-version: ${{ matrix.python-version }}
- name: Display Python version
run: python -c "import sys; print(sys.version)"
- name: Install flake8
run: python -m pip install flake8
- name: Check fMRIPrep
run: python -m flake8 fmriprep
- name: Check wrapper
run: python -m flake8 wrapper
16 changes: 0 additions & 16 deletions .travis.yml

This file was deleted.

5 changes: 2 additions & 3 deletions fmriprep/cli/parser.py
Original file line number Diff line number Diff line change
Expand Up @@ -635,9 +635,8 @@ def parse_args(args=None, namespace=None):
config.from_dict(vars(opts))

if not config.execution.notrack:
try:
import sentry_sdk
except ImportError:
import pkgutil
if pkgutil.find_loader("sentry_sdk") is None:
config.execution.notrack = True
config.loggers.cli.warning("Telemetry disabled because sentry_sdk is not installed.")
else:
Expand Down

0 comments on commit 442e21e

Please sign in to comment.