Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

move PRs with conflict to TO REBASE column #55

Merged
merged 5 commits into from
Sep 30, 2022
Merged

Conversation

jakubgs
Copy link
Member

@jakubgs jakubgs commented Jul 27, 2022

@jakubgs jakubgs requested a review from Marudny July 27, 2022 14:14
@jakubgs jakubgs self-assigned this Jul 27, 2022
jakubgs added a commit to status-im/status-mobile that referenced this pull request Jul 27, 2022
jakubgs added a commit to status-im/status-mobile that referenced this pull request Jul 27, 2022
jakubgs added a commit to status-im/status-mobile that referenced this pull request Jul 27, 2022
@jakubgs
Copy link
Member Author

jakubgs commented Jul 29, 2022

Hmmm, I have deployed this, but I have yet to see an example of moving a PR to TO REBASE column:

[email protected]:~ % zgrep 'Moved card' /var/log/docker/status-github-bot.web.1/docker.* | awk -F'INFO' '{print $2}'
 probot: assign-approved-pr-to-test - Moved card 81973280 to CONTRIBUTOR for PR #13360
 probot: assign-approved-pr-to-test - Moved card 84133355 to CONTRIBUTOR for PR #13660
 probot: assign-approved-pr-to-test - Moved card 84365417 to CONTRIBUTOR for PR #13698
 probot: assign-approved-pr-to-test - Moved card 84365417 to CONTRIBUTOR for PR #13698
 probot: assign-approved-pr-to-test - Moved card 84430279 to CONTRIBUTOR for PR #13707
 probot: assign-approved-pr-to-test - Moved card 83730887 to CONTRIBUTOR for PR #13602

So I might have to create a situation in which this move is triggered myself to test this.

@jakubgs jakubgs force-pushed the move-dirty-to-rebase branch 2 times, most recently from faeea1f to a19a42a Compare September 30, 2022 09:23
Otherwise we get:
```
DEPRECATED (@octokit/rest): `projects.moveProjectCard()` is deprecated, use `projects.moveCard()`
DEPRECATED (@octokit/rest): "id" parameter has been renamed to "card_id"
```

Signed-off-by: Jakub Sokołowski <[email protected]>
```
DEPRECATED (@octokit/rest): `issues.getForRepo()` is deprecated, use `issues.listForRepo()`
```

Signed-off-by: Jakub Sokołowski <[email protected]>
@jakubgs jakubgs merged commit c4d6664 into master Sep 30, 2022
@jakubgs jakubgs deleted the move-dirty-to-rebase branch September 30, 2022 11:59
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