Skip to content

Claude/fix main auto push zglf m#44

Merged
DaLuSt merged 4 commits intodevfrom
claude/fix-main-auto-push-ZglfM
Mar 7, 2026
Merged

Claude/fix main auto push zglf m#44
DaLuSt merged 4 commits intodevfrom
claude/fix-main-auto-push-ZglfM

Conversation

@DaLuSt
Copy link
Copy Markdown
Owner

@DaLuSt DaLuSt commented Mar 7, 2026

No description provided.

claude and others added 4 commits February 28, 2026 11:24
Runs pytest on every PR and push to any of the three long-lived
branches. This is the status check to require in branch protection
rules once the dev and testing branches are created on GitHub.

https://claude.ai/code/session_014HLhrFtCVRFnEyfRexiy3d
Add CI workflow gating PRs to main, testing, and dev
When a push lands on the `testing` branch, the new workflow:
1. Runs ruff lint and pytest tests
2. Checks whether testing is ahead of main
3. Creates a PR (testing → main) if one does not already exist
4. Merges the PR immediately using the built-in GITHUB_TOKEN

No extra secrets are required; the testing branch is preserved after merge.

https://claude.ai/code/session_01YbWdZzfzmkn5sqW3ByjUB7
Three issues were causing every CI run to fail:

1. web/app.py: remove unused `import time` (ruff F401)
2. tests/requirements.txt: add missing flask-wtf dependency required
   by test_web_app.py (importing web/app.py which uses CSRFProtect)
3. tests/test_web_app.py: set WTF_CSRF_ENABLED=False in the test
   client fixture so POST requests to /api/delete aren't blocked
   by CSRF validation (was returning 400 instead of 200/401)

All 26 tests now pass and ruff reports no issues.

https://claude.ai/code/session_01YbWdZzfzmkn5sqW3ByjUB7
@DaLuSt DaLuSt merged commit 69d1659 into dev Mar 7, 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.

2 participants