-
Notifications
You must be signed in to change notification settings - Fork 0
Cleanup #3
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
Cleanup #3
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
This file was deleted.
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,4 +1,4 @@ | ||
| FROM python:3.13-alpine | ||
| FROM python:3.11-alpine | ||
| RUN mkdir /app | ||
| WORKDIR /app | ||
| COPY requirements.txt . | ||
|
|
||
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
| @@ -0,0 +1,75 @@ | ||||||
| # | ||||||
| # Pipfile for managing project dependencies | ||||||
| # | ||||||
| # References: | ||||||
| # - https://pipenv.pypa.io/en/latest/pipfile.html | ||||||
| # | ||||||
| [[source]] | ||||||
| # https://pipenv.pypa.io/en/latest/pipfile.html#packages-section | ||||||
| url = "https://pypi.org/simple" | ||||||
| verify_ssl = true | ||||||
| name = "pypi" | ||||||
|
|
||||||
| [scripts] | ||||||
| # https://pipenv.pypa.io/en/latest/pipfile.html#packages-section | ||||||
|
|
||||||
| [packages] | ||||||
| # https://pipenv.pypa.io/en/latest/pipfile.html#packages-section | ||||||
| six = "*" | ||||||
| appdirs = "*" | ||||||
|
|
||||||
| [dev-packages] | ||||||
| # https://pipenv.pypa.io/en/latest/pipfile.html#packages-section | ||||||
| mypy = "*" | ||||||
| flake8 = "*" | ||||||
| pytest = "*" | ||||||
| black = "*" | ||||||
| isort = "*" | ||||||
| check-manifest = "*" | ||||||
| coverage = "*" | ||||||
| pycodestyle = "*" | ||||||
| pytest = "*" | ||||||
|
||||||
| pytest = "*" |
Copilot
AI
Feb 6, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is a typo in line 68: "alphabeticallyy" should be "alphabetically" (with only one 'y' at the end).
| sort_pipfile = true # Sort packages alphabeticallyy | |
| sort_pipfile = true # Sort packages alphabetically |
Copilot
AI
Feb 6, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The python_version should specify a major.minor version (e.g., "3.11"), not a patch version (e.g., "3.11.14"). Pipenv's python_version field is intended for specifying the major.minor Python version, not the patch level. The patch version specification may cause issues with Pipenv's version matching.
| python_version = "3.11.14" | |
| python_version = "3.11" |
Large diffs are not rendered by default.
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,21 @@ | ||
| # Updated by depupdate.py on 2026-02-05T22:27:06 by user | ||
| # Do not edit this section manually. | ||
| mypy==1.19.1 | ||
| flake8==7.3.0 | ||
| pytest==9.0.2 | ||
| black==26.1.0 | ||
| isort==7.0.0 | ||
| check-manifest==0.51 | ||
| coverage==7.13.3 | ||
| pycodestyle==2.14.0 | ||
| pytest==9.0.2 | ||
| pytest-cov==7.0.0 | ||
| pytest-html==4.2.0 | ||
| pytest-json-report==1.5.0 | ||
| tox==4.34.1 | ||
| tox-travis==0.13 | ||
| twine==6.2.0 | ||
| wheel==0.46.3 | ||
| sphinx==9.0.4 | ||
| sphinxcontrib-napoleon==0.7 | ||
| guzzle_sphinx_theme==0.7.11 |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,4 +0,0 @@ | ||
| # Updated by depupdate.py on 2025-05-29T00:19:36 by user | ||
| # Do not edit this section manually. | ||
| six==1.17.0 | ||
| appdirs==1.4.4 | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -8,55 +8,58 @@ | |
| import logging | ||
| import sys | ||
| from datetime import datetime | ||
| from typing import Any, Dict, Optional | ||
|
|
||
| from .__version__ import __title__ | ||
|
|
||
| # DATETIME CONSTANTS | ||
| EPOCH = datetime(1970, 1, 1) | ||
| ISO_DATETIME_STRING = "1970-01-01 00:00:00.000" | ||
| ISO_DATETIME_FORMAT = "%Y-%m-%d %H:%M:%S.%f" | ||
| EPOCH: datetime = datetime(1970, 1, 1) | ||
| ISO_DATETIME_STRING: str = "1970-01-01 00:00:00.000" | ||
| ISO_DATETIME_FORMAT: str = "%Y-%m-%d %H:%M:%S.%f" | ||
|
|
||
| # CONNECTION DEFAULTS | ||
| DEFAULT_MAX_POOL_CONNECTIONS = 10 | ||
| DEFAULT_RETRIES = 0 | ||
| DEFAULT_POOL_TIMEOUT = None | ||
| DEFAULT_POOLBLOCK = False | ||
| DEFAULT_POOLSIZE = 10 | ||
| DEFAULT_TIMEOUT = 60 | ||
| DEFAULT_MAX_POOL_CONNECTIONS: int = 10 | ||
| DEFAULT_RETRIES: int = 0 | ||
| DEFAULT_POOL_TIMEOUT: Optional[int] = None | ||
| DEFAULT_POOLBLOCK: bool = False | ||
| DEFAULT_POOLSIZE: int = 10 | ||
| DEFAULT_TIMEOUT: int = 60 | ||
|
|
||
| # FILE DEFAULTS | ||
| DEFAULT_CHUNK_SIZE = 64 * 2 ** 10 | ||
| DEFAULT_FILE_MODE_SUFFIX = "b" if sys.version_info[0] == 2 else "" | ||
| DEFAULT_FILE_WRITE_MODE = "w{0}".format(DEFAULT_FILE_MODE_SUFFIX) | ||
| DEFAULT_FILE_READ_MODE = "r{0}".format(DEFAULT_FILE_MODE_SUFFIX) | ||
| DEFAULT_CHUNK_SIZE: int = 64 * 2 ** 10 | ||
| DEFAULT_FILE_MODE_SUFFIX: str = "b" if sys.version_info[0] == 2 else "" | ||
|
||
| DEFAULT_FILE_WRITE_MODE: str = "w{0}".format(DEFAULT_FILE_MODE_SUFFIX) | ||
| DEFAULT_FILE_READ_MODE: str = "r{0}".format(DEFAULT_FILE_MODE_SUFFIX) | ||
|
|
||
| # LOGGING DEFAULTS | ||
| DEFAULT_LOGGER_NAME = __title__ | ||
| DEFAULT_LOGGER_NAME: str = __title__ | ||
|
|
||
| # LOGGING OPTIONS | ||
| LOGGING_DATEFMT = "%Y-%m-%d %H:%M:%S" | ||
| LOGGING_FILEMODE = "a+" | ||
| LOGGING_FILENAME = None | ||
| LOGGING_FORMAT = ( | ||
| LOGGING_DATEFMT: str = "%Y-%m-%d %H:%M:%S" | ||
| LOGGING_FILEMODE: str = "a+" | ||
| LOGGING_FILENAME: Optional[str] = None | ||
| LOGGING_FORMAT: str = ( | ||
| "(%(asctime)s) [%(levelname)s] " | ||
| "%(name)s.%(funcName)s(%(lineno)d): %(message)s" | ||
| ) | ||
| LOGGING_LEVEL = logging.ERROR | ||
| LOGGING_STYLE = "%" | ||
| LOGGING_LEVELS = { | ||
| LOGGING_LEVEL: int = logging.ERROR | ||
| LOGGING_STYLE: str = "%" | ||
|
|
||
| LOGGING_LEVELS: Dict[int, str] = { | ||
| logging.NOTSET: "sample", | ||
| logging.DEBUG: "debug", | ||
| logging.INFO: "info", | ||
| logging.WARNING: "warning", | ||
| logging.ERROR: "error", | ||
| logging.FATAL: "fatal", | ||
| } | ||
| LOGGING_LEVELS_MAP = { | ||
|
|
||
| LOGGING_LEVELS_MAP: Dict[str, int] = { | ||
| LOGGING_LEVELS[lvl]: lvl | ||
| for lvl in LOGGING_LEVELS | ||
| } | ||
| LOGGING_DICT = { | ||
| # TODO: implement test(s) | ||
|
|
||
| LOGGING_DICT: Dict[str, Any] = { | ||
| "version": 1, | ||
| "disable_existing_loggers": False, | ||
| "formatters": { | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The Dockerfile still references requirements.txt at lines 4-5, but this file is being deleted in this PR. The Dockerfile needs to be updated to use the new requirements-dev.txt file or another dependency management approach (such as Pipfile/Pipfile.lock), otherwise the Docker build will fail.