Skip to content

Conversation

dagitses
Copy link
Collaborator

@dagitses dagitses commented Mar 27, 2023

See D44409928.

Differential Revision: [D44429769](https://our.internmc.facebook.com/intern/diff/D44429769/)

[ghstack-poisoned]
@pytorch-bot
Copy link

pytorch-bot bot commented Mar 27, 2023

dagitses pushed a commit that referenced this pull request Mar 27, 2023
Pull Request resolved: #97690

See D44409928.
ghstack-source-id: 184149248

Differential Revision: [D44429769](https://our.internmc.facebook.com/intern/diff/D44429769/)
@lezcano lezcano removed their request for review March 27, 2023 20:56
@dagitses dagitses marked this pull request as ready for review March 28, 2023 02:34
mikey dagitses added 2 commits March 28, 2023 03:12
@dagitses dagitses requested review from ezyang and removed request for H-Huang, awgu, d4l3k, fegin, kiukchung, kwen2501, rohan-varma, wanchaol and zhaojuanmao April 6, 2023 20:35
@dagitses
Copy link
Collaborator Author

dagitses commented Apr 7, 2023

@pytorchbot --help

@pytorch-bot
Copy link

pytorch-bot bot commented Apr 7, 2023

PyTorchBot Help

usage: @pytorchbot [-h] {merge,revert,rebase,label,drci} ...

In order to invoke the bot on your PR, include a line that starts with
@pytorchbot anywhere in a comment. That line will form the command; no
multi-line commands are allowed. 

Example:
    Some extra context, blah blah, wow this PR looks awesome

    @pytorchbot merge

optional arguments:
  -h, --help            Show this help message and exit.

command:
  {merge,revert,rebase,label,drci}
    merge               Merge a PR
    revert              Revert a PR
    rebase              Rebase a PR
    label               Add label to a PR
    drci                Update Dr. CI

Merge

usage: @pytorchbot merge [-f MESSAGE | -ic] [-r [{viable/strict,master}]]

Merge an accepted PR, subject to the rules in .github/merge_rules.json.
By default, this will wait for all required checks (lint, pull) to succeed before merging.

optional arguments:
  -f MESSAGE, --force MESSAGE
                        Merge without checking anything. This requires a reason for auditting purpose, for example:
                        @pytorchbot merge -f 'Minor update to fix lint. Expecting all PR tests to pass'
  -ic, --ignore-current
                        Merge while ignore the currently failing jobs.  If there are no pending checks, use -f/--force since this will fail.
  -r [{viable/strict,master}], --rebase [{viable/strict,master}]
                        Rebase the PR to re run checks before merging.  Accepts viable/strict or master as branch options and will default to viable/strict if not specified.

Revert

usage: @pytorchbot revert -m MESSAGE -c
                          {nosignal,ignoredsignal,landrace,weird,ghfirst}

Revert a merged PR. This requires that you are a Meta employee.

Example:
  @pytorchbot revert -m="This is breaking tests on trunk. hud.pytorch.org/" -c=nosignal

optional arguments:
  -m MESSAGE, --message MESSAGE
                        The reason you are reverting, will be put in the commit message. Must be longer than 3 words.
  -c {nosignal,ignoredsignal,landrace,weird,ghfirst}, --classification {nosignal,ignoredsignal,landrace,weird,ghfirst}
                        A machine-friendly classification of the revert reason.

Rebase

usage: @pytorchbot rebase [-s | -b BRANCH]

Rebase a PR. Rebasing defaults to the stable viable/strict branch of pytorch.
You must have write permissions to the repo to rebase a PR.

optional arguments:
  -s, --stable          [DEPRECATED] Rebase onto viable/strict
  -b BRANCH, --branch BRANCH
                        Branch you would like to rebase to

Label

usage: @pytorchbot label labels [labels ...]

Adds label to a PR

positional arguments:
  labels  Labels to add to given Pull Request

Dr CI

usage: @pytorchbot drci 

Update Dr. CI. Updates the Dr. CI comment on the PR in case it's gotten out of sync with actual CI results.

@dagitses
Copy link
Collaborator Author

dagitses commented Apr 7, 2023

@pytorchbot drci

@facebook-github-bot
Copy link
Contributor

@pytorchbot merge -f 'Landed internally'

(Initiating merge automatically since Phabricator Diff has merged, using force because this PR might not pass merge_rules.json but landed internally)

@pytorchmergebot
Copy link
Collaborator

Merge started

Your change will be merged immediately since you used the force (-f) flag, bypassing any CI checks (ETA: 1-5 minutes).

Learn more about merging in the wiki.

Questions? Feedback? Please reach out to the PyTorch DevX Team

Advanced Debugging
Check the merge workflow status
here

@dagitses dagitses deleted the gh/dagitses/28/head branch April 8, 2023 02:03
huydhn added a commit to huydhn/pytorch that referenced this pull request Apr 8, 2023
pytorchmergebot pushed a commit that referenced this pull request Apr 8, 2023
#97690 breaks MacOS build https://hud.pytorch.org/pytorch/pytorch/commit/c68a94c5ead758cb0b3fa8a71d731fe131228b12.  The fix looks easy enough so I try to go ahead with a forward fix.
Pull Request resolved: #98665
Approved by: https://github.com/dagitses
pytorchmergebot pushed a commit to crcrpar/pytorch that referenced this pull request Apr 8, 2023
pytorchmergebot pushed a commit to crcrpar/pytorch that referenced this pull request Apr 8, 2023
skotapati pushed a commit to kulinseth/pytorch that referenced this pull request Apr 10, 2023
skotapati pushed a commit to kulinseth/pytorch that referenced this pull request Apr 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants