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

Updates Github workflows and minor code quality enhancements #4

Merged
merged 12 commits into from
Feb 11, 2025

Conversation

drache42
Copy link
Owner

@drache42 drache42 commented Feb 11, 2025

This pull request includes several changes to the GitHub workflows, configuration files, and codebase to improve the CI pipeline, linting, and code quality. The most important changes are grouped by theme below:

GitHub Workflows

  • Added a new linter.yml workflow to run Ruff linter on the codebase. This workflow includes setting up Python, installing dependencies with Poetry, and running the linter.
  • Updated the main.yml workflow to use the new linter.yml workflow for linting and to run unit tests across multiple Python versions.
  • Modified the unit-tests.yml workflow to accept a matrix input for Python versions and updated the steps to use the latest versions of actions and Poetry installation.

Configuration Files

  • Added filterwarnings = ["error"] to the pyproject.toml file to treat warnings as errors during tests.

Codebase

  • Suppressed InsecureRequestWarning from urllib3 in the check_url_status function to avoid unnecessary warnings during URL checks.

@drache42 drache42 changed the title Draft: Add CI workflows for linting and testing with Python versions Updates Github workflows and minor code quality enhancements Feb 11, 2025
@drache42 drache42 merged commit 25aaac2 into main Feb 11, 2025
6 checks passed
@drache42 drache42 deleted the dev/integration-tests branch February 11, 2025 19:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant