Skip to content

ci: Add GitHub Actions test workflow#25

Merged
rdmueller merged 4 commits intomainfrom
feature/ci-test-workflow
Jan 21, 2026
Merged

ci: Add GitHub Actions test workflow#25
rdmueller merged 4 commits intomainfrom
feature/ci-test-workflow

Conversation

@raifdmueller
Copy link
Collaborator

Summary

Adds automated test execution on Pull Requests via GitHub Actions.

Changes

  • Add .github/workflows/test.yml workflow
  • Tests run on every PR and push to main
  • Uses uv for dependency management
  • Runs ruff check for linting
  • Runs pytest for tests
  • Fix: Pass attributes parameter to _parse_elements method (was causing ruff F821 error)

Workflow Steps

  1. Checkout code
  2. Install uv via astral-sh/setup-uv@v4
  3. Install Python 3.12
  4. Install dependencies with uv sync
  5. Run linting with uv run ruff check
  6. Run tests with uv run pytest

Test plan

  • Tests pass locally (70 tests)
  • Linting passes locally
  • GitHub Actions workflow runs successfully on this PR

Closes #23

🤖 Generated with Claude Code

- Add .github/workflows/test.yml for automated testing on PRs
- Run pytest and ruff check on every PR and push to main
- Uses uv for dependency management
- Fix: Pass attributes parameter to _parse_elements method

Closes #23

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds automated CI testing via GitHub Actions and fixes a linting error in the AsciiDoc parser. The workflow runs on every push to main and on pull requests, executing both linting and test suites.

Changes:

  • Add GitHub Actions workflow for automated testing with uv dependency management
  • Fix bug where attributes parameter wasn't passed to _parse_elements method (ruff F821 error)
  • Improve code style with proper import ordering and whitespace cleanup in tests

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
.github/workflows/test.yml New CI workflow that runs ruff linting and pytest on PRs and main branch pushes
src/mcp_server/asciidoc_parser.py Fixes missing attributes parameter in _parse_elements call and removes problematic locals() check
tests/test_asciidoc_parser.py Reorders imports to follow Python conventions and removes trailing whitespace

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

rdmueller and others added 3 commits January 21, 2026 09:49
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
uv sync --dev doesn't exist, need --extra dev for optional-dependencies

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@rdmueller rdmueller merged commit ea6dcc5 into main Jan 21, 2026
3 checks passed
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.

CI: Automated Test Execution on Pull Requests

3 participants