Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add format GitHub workflow #671

Closed
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
14 changes: 14 additions & 0 deletions .github/workflows/black.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,14 @@
name: Lint

on: [push, pull_request]

jobs:
lint:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v3
- uses: psf/black@stable
with:
options: "--check --verbose"
src: "./src"
version: "~= 23.0"
3 changes: 1 addition & 2 deletions src/hgvs/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -49,8 +49,7 @@

"""

from __future__ import (absolute_import, division, print_function,
unicode_literals)
from __future__ import absolute_import, division, print_function, unicode_literals

import logging
import re
Expand Down
8 changes: 2 additions & 6 deletions src/hgvs/alignmentmapper.py
Original file line number Diff line number Diff line change
Expand Up @@ -27,17 +27,15 @@
#


from __future__ import (absolute_import, division, print_function,
unicode_literals)
from __future__ import absolute_import, division, print_function, unicode_literals

from bioutils.coordinates import strand_int_to_pm
from six.moves import range

import hgvs.location
from hgvs import global_config
from hgvs.enums import Datum
from hgvs.exceptions import (HGVSDataNotAvailableError, HGVSError,
HGVSInvalidIntervalError, HGVSUsageError)
from hgvs.exceptions import HGVSDataNotAvailableError, HGVSError, HGVSInvalidIntervalError, HGVSUsageError
from hgvs.utils import build_tx_cigar
from hgvs.utils.cigarmapper import CIGARMapper

Expand Down Expand Up @@ -88,7 +86,6 @@ def __init__(self, hdp, tx_ac, alt_ac, alt_aln_method):
self.alt_aln_method = alt_aln_method

if self.alt_aln_method != "transcript":

tx_info = hdp.get_tx_info(self.tx_ac, self.alt_ac, self.alt_aln_method)
if tx_info is None:
raise HGVSDataNotAvailableError(
Expand Down Expand Up @@ -126,7 +123,6 @@ def __init__(self, hdp, tx_ac, alt_ac, alt_aln_method):
self.tgt_len = self.cigarmapper.tgt_len

else:

# this covers the identity cases n <-> c
tx_identity_info = hdp.get_tx_identity_info(self.tx_ac)
if tx_identity_info is None:
Expand Down
11 changes: 4 additions & 7 deletions src/hgvs/assemblymapper.py
Original file line number Diff line number Diff line change
@@ -1,15 +1,12 @@
# -*- coding: utf-8 -*-

from __future__ import (absolute_import, division, print_function,
unicode_literals)
from __future__ import absolute_import, division, print_function, unicode_literals

import logging

import hgvs
import hgvs.normalizer
from hgvs.exceptions import (HGVSDataNotAvailableError, HGVSError,
HGVSInvalidVariantError,
HGVSUnsupportedOperationError)
from hgvs.exceptions import HGVSDataNotAvailableError, HGVSError, HGVSInvalidVariantError, HGVSUnsupportedOperationError
from hgvs.variantmapper import VariantMapper

_logger = logging.getLogger(__name__)
Expand Down Expand Up @@ -57,7 +54,7 @@ def __init__(
replace_reference=hgvs.global_config.mapping.replace_reference,
add_gene_symbol=hgvs.global_config.mapping.add_gene_symbol,
*args,
**kwargs
**kwargs,
):
"""
:param object hdp: instance of hgvs.dataprovider subclass
Expand All @@ -78,7 +75,7 @@ def __init__(
prevalidation_level=prevalidation_level,
add_gene_symbol=add_gene_symbol,
*args,
**kwargs
**kwargs,
)
self.assembly_name = assembly_name
self.alt_aln_method = alt_aln_method
Expand Down
3 changes: 1 addition & 2 deletions src/hgvs/config.py
Original file line number Diff line number Diff line change
Expand Up @@ -27,8 +27,7 @@

"""

from __future__ import (absolute_import, division, print_function,
unicode_literals)
from __future__ import absolute_import, division, print_function, unicode_literals

import logging
import re
Expand Down
3 changes: 1 addition & 2 deletions src/hgvs/dataproviders/interface.py
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,7 @@

"""

from __future__ import (absolute_import, division, print_function,
unicode_literals)
from __future__ import absolute_import, division, print_function, unicode_literals

import abc
import os
Expand Down
3 changes: 0 additions & 3 deletions src/hgvs/dataproviders/ncbi.py
Original file line number Diff line number Diff line change
Expand Up @@ -311,7 +311,6 @@ def _get_cursor(self, n_retries=1):
n_tries_rem = n_retries + 1
while n_tries_rem > 0:
try:

conn = self._pool.getconn() if self.pooling else self._conn

# autocommit=True obviates closing explicitly
Expand All @@ -330,7 +329,6 @@ def _get_cursor(self, n_retries=1):
break

except psycopg2.OperationalError:

_logger.warning("Lost connection to {url}; attempting reconnect".format(url=self.url))
if self.pooling:
self._pool.closeall()
Expand All @@ -340,7 +338,6 @@ def _get_cursor(self, n_retries=1):
n_tries_rem -= 1

else:

# N.B. Probably never reached
raise HGVSError("Permanently lost connection to {url} ({n} retries)".format(url=self.url, n=n_retries))

Expand Down
3 changes: 1 addition & 2 deletions src/hgvs/dataproviders/seqfetcher.py
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,7 @@

"""

from __future__ import (absolute_import, division, print_function,
unicode_literals)
from __future__ import absolute_import, division, print_function, unicode_literals

import logging
import os
Expand Down
6 changes: 1 addition & 5 deletions src/hgvs/dataproviders/uta.py
Original file line number Diff line number Diff line change
Expand Up @@ -4,8 +4,7 @@

"""

from __future__ import (absolute_import, division, print_function,
unicode_literals)
from __future__ import absolute_import, division, print_function, unicode_literals

import contextlib
import inspect
Expand Down Expand Up @@ -580,7 +579,6 @@ def _get_cursor(self, n_retries=1):
n_tries_rem = n_retries + 1
while n_tries_rem > 0:
try:

conn = self._pool.getconn() if self.pooling else self._conn

# autocommit=True obviates closing explicitly
Expand All @@ -604,7 +602,6 @@ def _get_cursor(self, n_retries=1):
break

except psycopg2.OperationalError:

_logger.warning("Lost connection to {url}; attempting reconnect".format(url=self.url))
if self.pooling:
self._pool.putconn(conn)
Expand All @@ -616,7 +613,6 @@ def _get_cursor(self, n_retries=1):
n_tries_rem -= 1

else:

# N.B. Probably never reached
raise HGVSError("Permanently lost connection to {url} ({n} retries)".format(url=self.url, n=n_retries))

Expand Down
3 changes: 1 addition & 2 deletions src/hgvs/decorators/deprecated.py
Original file line number Diff line number Diff line change
Expand Up @@ -4,8 +4,7 @@

"""

from __future__ import (absolute_import, division, print_function,
unicode_literals)
from __future__ import absolute_import, division, print_function, unicode_literals

import collections
import warnings
Expand Down
3 changes: 1 addition & 2 deletions src/hgvs/decorators/lru_cache.py
Original file line number Diff line number Diff line change
Expand Up @@ -11,8 +11,7 @@

"""

from __future__ import (absolute_import, division, print_function,
unicode_literals)
from __future__ import absolute_import, division, print_function, unicode_literals

from collections import namedtuple
from functools import update_wrapper
Expand Down
3 changes: 1 addition & 2 deletions src/hgvs/edit.py
Original file line number Diff line number Diff line change
Expand Up @@ -9,8 +9,7 @@

"""

from __future__ import (absolute_import, division, print_function,
unicode_literals)
from __future__ import absolute_import, division, print_function, unicode_literals

import attr
import six
Expand Down
3 changes: 1 addition & 2 deletions src/hgvs/exceptions.py
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,7 @@

"""

from __future__ import (absolute_import, division, print_function,
unicode_literals)
from __future__ import absolute_import, division, print_function, unicode_literals


class HGVSError(Exception):
Expand Down
3 changes: 1 addition & 2 deletions src/hgvs/hgvsposition.py
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,7 @@

"""

from __future__ import (absolute_import, division, print_function,
unicode_literals)
from __future__ import absolute_import, division, print_function, unicode_literals

import attr

Expand Down
3 changes: 1 addition & 2 deletions src/hgvs/intervalmapper.py
Original file line number Diff line number Diff line change
Expand Up @@ -42,8 +42,7 @@
start_i and minimal end_i.
"""

from __future__ import (absolute_import, division, print_function,
unicode_literals)
from __future__ import absolute_import, division, print_function, unicode_literals

import logging
import re
Expand Down
6 changes: 2 additions & 4 deletions src/hgvs/location.py
Original file line number Diff line number Diff line change
Expand Up @@ -15,8 +15,7 @@
* :class:`Interval` -- an interval of Positions
"""

from __future__ import (absolute_import, division, print_function,
unicode_literals)
from __future__ import absolute_import, division, print_function, unicode_literals

from functools import total_ordering

Expand All @@ -25,8 +24,7 @@

import hgvs
from hgvs.enums import Datum, ValidationLevel
from hgvs.exceptions import (HGVSInvalidIntervalError,
HGVSUnsupportedOperationError)
from hgvs.exceptions import HGVSInvalidIntervalError, HGVSUnsupportedOperationError


@attr.s(slots=True, repr=False, cmp=False)
Expand Down
7 changes: 2 additions & 5 deletions src/hgvs/normalizer.py
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,7 @@
"""hgvs.normalizer
"""

from __future__ import (absolute_import, division, print_function,
unicode_literals)
from __future__ import absolute_import, division, print_function, unicode_literals

import copy
import logging
Expand All @@ -13,9 +12,7 @@
import hgvs
import hgvs.validator
import hgvs.variantmapper
from hgvs.exceptions import (HGVSDataNotAvailableError,
HGVSInvalidVariantError,
HGVSUnsupportedOperationError)
from hgvs.exceptions import HGVSDataNotAvailableError, HGVSInvalidVariantError, HGVSUnsupportedOperationError
from hgvs.utils.norm import normalize_alleles

_logger = logging.getLogger(__name__)
Expand Down
4 changes: 2 additions & 2 deletions src/hgvs/parser.py
Original file line number Diff line number Diff line change
Expand Up @@ -4,8 +4,7 @@

"""

from __future__ import (absolute_import, division, print_function,
unicode_literals)
from __future__ import absolute_import, division, print_function, unicode_literals

import copy
import logging
Expand All @@ -17,6 +16,7 @@
from pkg_resources import resource_filename

import hgvs.edit

# The following imports are referenced by fully-qualified name in the
# hgvs grammar.
import hgvs.enums
Expand Down
3 changes: 1 addition & 2 deletions src/hgvs/posedit.py
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,7 @@

"""

from __future__ import (absolute_import, division, print_function,
unicode_literals)
from __future__ import absolute_import, division, print_function, unicode_literals

import attr

Expand Down
3 changes: 1 addition & 2 deletions src/hgvs/projector.py
Original file line number Diff line number Diff line change
Expand Up @@ -4,8 +4,7 @@

"""

from __future__ import (absolute_import, division, print_function,
unicode_literals)
from __future__ import absolute_import, division, print_function, unicode_literals

import copy

Expand Down
3 changes: 1 addition & 2 deletions src/hgvs/sequencevariant.py
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,7 @@

"""

from __future__ import (absolute_import, division, print_function,
unicode_literals)
from __future__ import absolute_import, division, print_function, unicode_literals

import re

Expand Down
45 changes: 37 additions & 8 deletions src/hgvs/shell.py
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,7 @@
"""start IPython shell with hgvs initialized. Intended to be used for
experimenting, debugging, and generating bug reports."""

from __future__ import (absolute_import, division, print_function,
unicode_literals)
from __future__ import absolute_import, division, print_function, unicode_literals

import logging
import os
Expand Down Expand Up @@ -46,12 +45,42 @@ def shell():
logging.basicConfig(level=os.environ.get("HGVS_LOGGING_LEVEL", logging.WARNING))

from hgvs.easy import ( # noqa: F401; instances; functionalized methods
__version__, am37, am38, c_to_g, c_to_n, c_to_p, g_to_c, g_to_n,
g_to_t, get_relevant_transcripts, global_config, hdp,
hgvs_assembly_mapper_37, hgvs_assembly_mapper_38, hgvs_data_provider,
hgvs_normalizer, hgvs_parser, hgvs_validator, hgvs_variant_mapper, hn,
hp, hv, n_to_c, n_to_g, normalize, normalizer, parse, parser,
projector, t_to_g, t_to_p, validate, validator, variant_mapper, vm)
__version__,
am37,
am38,
c_to_g,
c_to_n,
c_to_p,
g_to_c,
g_to_n,
g_to_t,
get_relevant_transcripts,
global_config,
hdp,
hgvs_assembly_mapper_37,
hgvs_assembly_mapper_38,
hgvs_data_provider,
hgvs_normalizer,
hgvs_parser,
hgvs_validator,
hgvs_variant_mapper,
hn,
hp,
hv,
n_to_c,
n_to_g,
normalize,
normalizer,
parse,
parser,
projector,
t_to_g,
t_to_p,
validate,
validator,
variant_mapper,
vm,
)
from hgvs.utils.context import variant_context_w_alignment # noqa

IPython.embed(
Expand Down
Loading