Skip to content

Commit

Permalink
Replace appdirs with platformdirs
Browse files Browse the repository at this point in the history
  • Loading branch information
jwodder committed Nov 8, 2023
1 parent 59b6e12 commit 0e58464
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 4 deletions.
4 changes: 2 additions & 2 deletions dandi/cli/command.py
Original file line number Diff line number Diff line change
Expand Up @@ -9,9 +9,9 @@
import sys
from types import SimpleNamespace

import appdirs
import click
from click_didyoumean import DYMGroup
import platformdirs

from .base import lgr, map_to_click_exceptions
from .. import __version__, set_logger_level
Expand Down Expand Up @@ -97,7 +97,7 @@ def main(ctx, log_level, pdb=False):
h.addFilter(lambda r: not getattr(r, "file_only", False))
set_logger_level(h, log_level)

logdir = appdirs.user_log_dir("dandi-cli", "dandi")
logdir = platformdirs.user_log_dir("dandi-cli", "dandi")

Check warning on line 100 in dandi/cli/command.py

View check run for this annotation

Codecov / codecov/patch

dandi/cli/command.py#L100

Added line #L100 was not covered by tests
logfile = os.path.join(
logdir, "{:%Y%m%d%H%M%SZ}-{}.log".format(datetime.utcnow(), os.getpid())
)
Expand Down
2 changes: 1 addition & 1 deletion setup.cfg
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,6 @@ project_urls =
[options]
python_requires = >=3.8
install_requires =
appdirs
bidsschematools ~= 0.7.0
click
click-didyoumean
Expand All @@ -44,6 +43,7 @@ install_requires =
keyring != 23.9.0
keyrings.alt
packaging
platformdirs
pycryptodomex # for EncryptedKeyring backend in keyrings.alt
pydantic >= 1.9.0
pynwb >= 1.0.3,!=1.1.0,!=2.3.0
Expand Down
1 change: 0 additions & 1 deletion tox.ini
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,6 @@ commands =
[testenv:typing]
deps =
mypy
types-appdirs
types-python-dateutil
types-requests
commands =
Expand Down

0 comments on commit 0e58464

Please sign in to comment.