Skip to content

Add type ignore lint to Makefile#56587

Closed
samestep wants to merge 1 commit intopytorch:masterfrom
samestep:type-ignore-local-lint
Closed

Add type ignore lint to Makefile#56587
samestep wants to merge 1 commit intopytorch:masterfrom
samestep:type-ignore-local-lint

Conversation

@samestep
Copy link
Copy Markdown
Contributor

Followup to #56290 which adds the new lint to the local runner from #56439.

Test plan:

Same as #56439.

@samestep samestep requested review from a team and driazati April 21, 2021 15:14
@facebook-github-bot
Copy link
Copy Markdown
Contributor

facebook-github-bot commented Apr 21, 2021

💊 CI failures summary and remediations

As of commit a87c75f (more details on the Dr. CI page):


None of the CI failures appear to be your fault 💚



❄️ 2 failures tentatively classified as flaky

but reruns have not yet been triggered to confirm:

See CircleCI build pytorch_ios_12_0_0_x86_64_build (1/2)

Step: "Spin up environment" (full log | diagnosis details | 🔁 rerun) ❄️

Initializing agent
Build-agent version 1.0.56676-c1aa2884 (2021-04-21T11:56:01+0000)
Creating a dedicated VM with xcode:12.0 image

Assigned VM request id: 37786300
  provisioning: ..
VM 'default-bbc45533-d19c-4360-80a4-68de12625331' has been created
Initializing agent


Unexpected environment preparation error: failed to upload runner binary: failed to scp: signal: killed

See CircleCI build pytorch_ios_12_0_0_x86_64_lite_interpreter_build (2/2)

Step: "Spin up environment" (full log | diagnosis details | 🔁 rerun) ❄️

Initializing agent
Build-agent version 1.0.56676-c1aa2884 (2021-04-21T11:56:01+0000)
Creating a dedicated VM with xcode:12.0 image

Assigned VM request id: 37787262
  provisioning: .
VM 'default-367a0deb-bbd8-41a7-a949-de9b5d5b28ab' has been created
Initializing agent


Unexpected environment preparation error: failed to upload runner binary: failed to scp: signal: killed


This comment was automatically generated by Dr. CI (expand for details).Follow this link to opt-out of these comments for your Pull Requests.

Please report bugs/suggestions to the (internal) Dr. CI Users group.

@facebook-github-bot
Copy link
Copy Markdown
Contributor

@samestep has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Copy Markdown
Contributor

@samestep merged this pull request in fe0e1c7.

heitorschueroff pushed a commit that referenced this pull request Apr 21, 2021
Summary:
Followup to #56290 which adds the new lint to the local runner from #56439.

Pull Request resolved: #56587

Test Plan: Same as #56439.

Reviewed By: walterddr

Differential Revision: D27909889

Pulled By: samestep

fbshipit-source-id: 8b67f3bc36c9b5567fe5a9e49904f2cf23a9f135
krshrimali pushed a commit to krshrimali/pytorch that referenced this pull request May 19, 2021
Summary:
Followup to pytorch#56290 which adds the new lint to the local runner from pytorch#56439.

Pull Request resolved: pytorch#56587

Test Plan: Same as pytorch#56439.

Reviewed By: walterddr

Differential Revision: D27909889

Pulled By: samestep

fbshipit-source-id: 8b67f3bc36c9b5567fe5a9e49904f2cf23a9f135
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.

3 participants