Skip to content

Add GitHub Actions workflow for Paketbox tests#17

Merged
AndreasBeyer merged 1 commit intomainfrom
codex/create-github-actions-workflow-for-unit-tests
Nov 16, 2025
Merged

Add GitHub Actions workflow for Paketbox tests#17
AndreasBeyer merged 1 commit intomainfrom
codex/create-github-actions-workflow-for-unit-tests

Conversation

@AndreasBeyer
Copy link
Owner

Summary

  • add a GitHub Actions workflow that sets up Python 3.12 and runs the Paketbox test suite on pushes to main and on pull requests

Testing

  • PYTHONPATH=. python3 tests/run_tests.py (fails: existing failures in the baseline test suite)

Codex Task

@AndreasBeyer AndreasBeyer merged commit e8df7ac into main Nov 16, 2025
1 check failed
@AndreasBeyer AndreasBeyer deleted the codex/create-github-actions-workflow-for-unit-tests branch November 16, 2025 20:34
Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +22 to +27
- name: Run test suite
env:
PYTHONPATH: .
run: |
python3 --version
python3 tests/run_tests.py

Choose a reason for hiding this comment

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

P1 Badge Avoid introducing permanently red CI job

The new workflow step at lines 22‑27 runs python3 tests/run_tests.py and fails whenever the suite is not green. At the current HEAD the command exits with seven failures in TestPaketBoxIntegration.test_simultaneous_door_operations, so this job will fail on every push to main and every pull request as soon as the workflow lands. Because the job is required and not marked continue-on-error, all future PRs will be blocked even though those failures are pre-existing. Please either fix the failing tests in the same change or make the job optional until the suite is green, otherwise the workflow cannot complete successfully for anyone.

Useful? React with 👍 / 👎.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant