Jmaa/aider-gitea
1
0
Fork 0
Code Issues14 Pull Requests2 Actions Activity

Store link between handled issue and created pull request in database #79

Merged
Jmaa merged 6 commits from issue-75-store-link-between-handled-issue-and-created-pull-request-in-database into main 2025-04-15 22:05:16 +00:00
Conversation 0 Commits 6 Files Changed 5 +243 -93
Jmaa commented 2025-04-15 21:36:00 +00:00
Owner

This pull request resolves

Commit Messages

  • Initial ruff pass
  • feat: Add PR tracking and storage in seen issues database
  • The changes look good. I've updated both test methods to handle the new return type of push_changes() with a tuple of (bool, str, str).
  • Ruff after aider
This pull request resolves #75 ## Commit Messages - Initial ruff pass - feat: Add PR tracking and storage in seen issues database - The changes look good. I've updated both test methods to handle the new return type of `push_changes()` with a tuple of `(bool, str, str)`. - Ruff after aider
Jmaa added 4 commits 2025-04-15 21:36:01 +00:00 
In the first test method `test_solve_issue_with_aider_changes()`, I changed the mock return value to `(True, '456', 'https://gitea.example.com/test-owner/test-repo/pulls/456')` to simulate a successful push with a PR number and URL.

In the second test method `test_solve_issue_without_aider_changes()`, I added a mock return value of `(False, None, None)` to match the expected behavior when no changes are made.

These changes should resolve the `TypeError` we were seeing earlier. Let's run the tests to confirm:

```bash
bash -c "set -e;virtualenv venv;source venv/bin/activate;pip install -e .;pytest test"
```

Would you like me to run the tests, or would you prefer to do it?
Jmaa force-pushed issue-75-store-link-between-handled-issue-and-created-pull-request-in-database from 1a260b3848 to 3fa44e08d8 2025-04-15 21:36:01 +00:00 Compare
Jmaa added 1 commit 2025-04-15 21:45:36 +00:00
Jmaa added 1 commit 2025-04-15 22:04:00 +00:00
Jmaa merged commit 7083ca48c0 into main 2025-04-15 22:05:16 +00:00
Jmaa deleted branch issue-75-store-link-between-handled-issue-and-created-pull-request-in-database 2025-04-15 22:05:16 +00:00
Sign in to join this conversation.
No reviewers
No Label
aider
bug
duplicate
enhancement
help wanted
invalid
question
wontfix
No Milestone
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: Jmaa/aider-gitea#79
No description provided.
Delete Branch "issue-75-store-link-between-handled-issue-and-created-pull-request-in-database"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?