Skip to content

github: add write methods (bug 1989960)#611

Merged
zzzeid merged 19 commits intozeid/bug-1989635-github-pr-pilotfrom
zeid/bug-1989960-modify-pr-api
Oct 28, 2025
Merged

github: add write methods (bug 1989960)#611
zzzeid merged 19 commits intozeid/bug-1989635-github-pr-pilotfrom
zeid/bug-1989960-modify-pr-api

Conversation

@zzzeid
Copy link
Copy Markdown
Contributor

@zzzeid zzzeid commented Oct 15, 2025

  • add methods to open, close, and comment on pull requests

@zzzeid zzzeid changed the base branch from zeid/bug-1989637-github-api-client to zeid/bug-1991125-landing-job-backend-frontend October 17, 2025 13:58
@zzzeid zzzeid force-pushed the zeid/bug-1989960-modify-pr-api branch 2 times, most recently from f93736f to c2c8043 Compare October 17, 2025 14:19
@zzzeid zzzeid force-pushed the zeid/bug-1991125-landing-job-backend-frontend branch from 464c6a0 to c9e89e8 Compare October 21, 2025 17:18
@zzzeid zzzeid force-pushed the zeid/bug-1989960-modify-pr-api branch from c2c8043 to f0f00ea Compare October 21, 2025 17:19
@zzzeid zzzeid force-pushed the zeid/bug-1991125-landing-job-backend-frontend branch from c9e89e8 to f53b84c Compare October 21, 2025 17:48
@zzzeid zzzeid force-pushed the zeid/bug-1989960-modify-pr-api branch from f0f00ea to 084b2a2 Compare October 21, 2025 17:49
@zzzeid zzzeid force-pushed the zeid/bug-1991125-landing-job-backend-frontend branch from f53b84c to f0579e6 Compare October 21, 2025 18:12
- add GitHubAPI class
- add GitHubAPIClient class
- move "get token" functionality to GitHubAPI class
- fix failing test_GitSCM_push_get_github_token
@zzzeid zzzeid force-pushed the zeid/bug-1991125-landing-job-backend-frontend branch 3 times, most recently from 85dade5 to eb53a97 Compare October 21, 2025 20:08
@zzzeid zzzeid force-pushed the zeid/bug-1989960-modify-pr-api branch 2 times, most recently from 3517928 to 947c93e Compare October 21, 2025 20:14
@zzzeid zzzeid requested a review from shtrom October 21, 2025 20:14
@zzzeid zzzeid marked this pull request as ready for review October 21, 2025 20:14
@zzzeid zzzeid force-pushed the zeid/bug-1991125-landing-job-backend-frontend branch from eb53a97 to 1c63a7a Compare October 21, 2025 20:19
@zzzeid zzzeid force-pushed the zeid/bug-1989960-modify-pr-api branch from 947c93e to dea1b83 Compare October 21, 2025 20:20
shtrom

This comment was marked as resolved.

Copy link
Copy Markdown
Member

@shtrom shtrom left a comment

Choose a reason for hiding this comment

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

Meh, the discussion doesn't need to block landing this. r+!

Base automatically changed from zeid/bug-1991125-landing-job-backend-frontend to zeid/bug-1989635-github-pr-pilot October 28, 2025 12:53
@zzzeid zzzeid merged commit a9a01e9 into zeid/bug-1989635-github-pr-pilot Oct 28, 2025
1 check passed
@zzzeid zzzeid deleted the zeid/bug-1989960-modify-pr-api branch October 28, 2025 13:16
zzzeid added a commit that referenced this pull request Oct 28, 2025
- add methods to open, close, and comment on pull requests
zzzeid added a commit that referenced this pull request Oct 28, 2025
- add methods to open, close, and comment on pull requests
shtrom pushed a commit that referenced this pull request Oct 31, 2025
- add methods to open, close, and comment on pull requests
zzzeid added a commit that referenced this pull request Nov 6, 2025
- add methods to open, close, and comment on pull requests
zzzeid added a commit that referenced this pull request Nov 6, 2025
- add methods to open, close, and comment on pull requests
zzzeid added a commit that referenced this pull request Nov 20, 2025
- add methods to open, close, and comment on pull requests
zzzeid added a commit that referenced this pull request Nov 24, 2025
- add methods to open, close, and comment on pull requests
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