refactor: simplify GitHub Actions workflows#45
Merged
Conversation
- Rename CI workflow to Tests (ci.yml → tests.yml) - Simplify PyPI publish: use uv build/publish instead of pypa action - Use ubuntu-slim for publish workflow (no longer needs Docker) - Replace explicit Python install with UV_PYTHON env var - Update actions/checkout to v6 - Update README badge URL
⸜(｡˃ ᵕ ˂ )⸝♡ Thank you for opening this Pull Request, AzisK!( ˶°ㅁ°) !! It's Trivia Time!Here are 3 trivia questions to keep you entertained while CI runs. 🧩 Q1: The theme for the popular science fiction series "Doctor Who" was composed by who?A) Ron Grainer 🧩 Q2: Who is Winnie the Pooh's gloomy donkey friend?A) Eeyore 🧩 Q3: How many sides does a trapezium have?A) 3
|
|
🎉 All tests passed! Here's a dog for you! 🐶 |
Tests for user config loading were flaky across Python versions because USER_CONFIG_PATH was imported at module level before pyfakefs was active. Some Python versions handle this correctly, others don't. Move imports of USER_CONFIG_PATH, load_user_categories_config, and DEFAULT_CATEGORIES inside each test method to ensure they resolve after pyfakefs is active, making tests consistent across all Python/OS combos.
- Split dev dependencies into test, lint, and dev groups - test: pytest, pyfakefs, ruff (for test job) - lint: mypy, types-tqdm (for mypy job) - dev: includes both + pre-commit (for local development) - Remove tests for Python 3.8
|
🎉 All tests passed! Here's a dog for you! 🐶 |
|
🎉 All tests passed! Here's a dog for you! 🐶 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
shutils.disk_usage(displays 0s)