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

Resolving 3.11 Build Issues #234

Merged
merged 6 commits into from
Mar 16, 2024
Merged
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
5 changes: 2 additions & 3 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -20,9 +20,8 @@ jobs:
with:
python-version: ${{ matrix.python-version }}
- name: Install poetry
uses: abatilo/actions-poetry@v2
with:
poetry-version: ${{ matrix.poetry-version }}
run: |
pipx install poetry=="1.7.1"
- name: Define a cache for the virtual environment based on the dependencies lock file
uses: actions/cache@v4
with:
Expand Down
5 changes: 2 additions & 3 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -14,9 +14,8 @@ jobs:
with:
python-version: "3.9.16"
- name: Install poetry
uses: abatilo/actions-poetry@v2
with:
poetry-version: "1.7.1"
run: |
pipx install poetry=="1.7.1"
- name: Define a cache for the virtual environment based on the dependencies lock file
uses: actions/cache@v4
with:
Expand Down
31 changes: 25 additions & 6 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -12,26 +12,45 @@ A Configurable and Context Driven Project 1999 Log Parser with NCurses Interface

#### PyPI
```sh
$ pip3 install eqalert
$ # or if your distribution is using externally managed environemnts
# Download
$ pipx install eqalert

# Start eqalert
$ eqalert
```

#### Locally
```sh
# Clone the repository
$ git clone https://github.com/mgeitz/eqalert.git
$ cd eqalert
$ python3 -m pip install -e .
$ # or if your distribution is using externally managed environments
$ # (remember the venv must be sourced to run eqalert when installed this way)
$ python3 -m venv .venv; source .venv/bin/activate; python3 -m pip install -e .

# Install poetry
$ pipx install poetry

# Install playsound
$ poetry run pip install --upgrade pip
$ poetry run pip install --upgrade wheel
$ poetry run pip install playsound

# Retrieve dependencies and build
$ poetry install
$ poetry build

# Start eqalert
$ poetry run eqalert
```

#### Docker
```sh
# Clone the repository
$ git clone https://github.com/mgeitz/eqalert.git
$ cd eqalert

# Build
$ docker compose build

# Start eqalert
$ docker compose run eqalert
```

Expand Down
2 changes: 1 addition & 1 deletion eqa/lib/config.py
Original file line number Diff line number Diff line change
Expand Up @@ -26607,7 +26607,7 @@ def write_config(base_path, config_name, version, new_config):
elif os.path.isfile(line_json_path):
### Validate file is readable

old_version = "unknown/"
old_version = "unknown"
generate_config = False
try:
json_data = open(line_json_path, "r", encoding="utf-8")
Expand Down
39 changes: 27 additions & 12 deletions poetry.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

8 changes: 5 additions & 3 deletions pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -10,14 +10,16 @@ packages = [
]

[tool.poetry.dependencies]
python = "~3.9.2"
python = ">=3.9.2"
playsound = "~1.3.0"
gTTS = "~2.3.2"

gtts = "~2.3.2"

[tool.poetry.group.dev.dependencies]
pytest = "^8.0.2"

[tool.poetry.scripts]
eqalert = 'eqa.eqalert:main'

[build-system]
requires = ["poetry-core"]
build-backend = "poetry.core.masonry.api"
2 changes: 1 addition & 1 deletion tests/test_dummy.py
Original file line number Diff line number Diff line change
@@ -1,2 +1,2 @@
def test_dummy():
assert True
assert True