[PATCH] D104981: [clang-tidy] Add -line-filter to run-clang-tidy.py

[Kirill Bobyrev via Phabricator via cfe-commits]

kbobyrev added a comment.

In D104981#2845005 <https://reviews.llvm.org/D104981#2845005>, @bansan wrote:

> @kbobyrev Yes please. I downloaded arc. I created a commit where last line is "Differential Revision: https://reviews.llvm.org/D104981".
>
> I linked my github account to llvm's phabricator.
>
> But when I `arc land`, I can't logon with my github username.
>
>    STRATEGY  Merging with "squash" strategy, the default strategy.
>    SOURCE  Landing the current branch, "main".
>    ONTO REMOTE  Remote "origin" was selected by following tracking branches upstream to the closest remote.
>    ONTO TARGET  Refs were selected by reading "arc.land.onto" configuration: main.
>    INTO REMOTE  Will merge into remote "origin" by default, because this is the remote the change is landing onto.
>    INTO TARGET  Will merge into target "main" by default, because this is the "onto" target.
>    FETCH  Fetching "main" from remote "origin"...
>   
>     $   git fetch --no-tags --quiet -- origin main
>   
>   
>    INTO COMMIT  Preparing merge into "main" from remote "origin", at commit "614b46e4dcab".
>    LANDING  These changes will land:
>   
>     *   D104981 [clang-tidy] Add -line-filter to run-clang-tidy.py
>             0610f15fab16  [clang-tidy] Add -line-filter to run-clang-tidy.py
>   
>    >>>  Land these changes? [y/N/?] y
>    MERGING  0610f15fab16 [clang-tidy] Add -line-filter to run-clang-tidy.py
>    MERGE  Attempting to rebase changes.
>    DONE  Merge succeeded.
>    PUSHING  Pushing changes to "origin".
>   
>     $   git push -- origin 6adf32fecc25:refs/heads/main
>   
>   Username for 'https://github.com': bansan85
>   Password for 'https://bansan85@github.com':
>   remote: Permission to llvm/llvm-project.git denied to bansan85.
>   fatal: unable to access 'https://github.com/llvm/llvm-project.git/': The requested URL returned error: 403
>   
>    LOAD STATE  Restoring local state (to ref "main" at commit "0610f15fab16").
>    USAGE EXCEPTION  Push failed! Fix the error and run "arc land" again.
>
> I try to follow https://github.com/llvm/llvm-project/blob/main/llvm/docs/Phabricator.rst but I don't know what I missed.
>
> Thanks,

Landed the patch!

I think the problem is that you do not have the commit access yet. Here's how to get it: https://llvm.org/docs/DeveloperPolicy.html#obtaining-commit-access


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D104981/new/

https://reviews.llvm.org/D104981