Skip to content

Commit

Permalink
chore(datafile): ignore lint rule for codacy, pending refactor (#2363)
Browse files Browse the repository at this point in the history
Codacy began detecting a "severe" pylint E1126 invalid sequence index violation in #2362.

The issue pre-existed that PR, I think the format changes just revealed it. We could just ignore it in the web UI but this is probably worth coming back to, to appease linters and type checkers and to make the code easier to reason about.

Relatedly, there is work ongoing to make ruff work with codacy, so hopefully in future we can use ruff ignore syntax for things like this if/when needed
  • Loading branch information
wpbonelli authored Nov 12, 2024
1 parent a5545c6 commit dc394cd
Showing 1 changed file with 9 additions and 0 deletions.
9 changes: 9 additions & 0 deletions flopy/utils/datafile.py
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,15 @@
"""

# in LayerFile, the recordarray attribute begins its life as
# a list, which is appended to in subclasses' build_index(),
# then finally becomes an array, after which it's accessed
# in this file by column name. this probably deserves some
# attention, but in the meantime, disable the pylint rule
# to appease codacy.
#
# pylint: disable=invalid-sequence-index

import os
import warnings
from pathlib import Path
Expand Down

0 comments on commit dc394cd

Please sign in to comment.