Skip to content

Conversation

samestep
Copy link
Contributor

This should clarify its purpose, which is:

to make sure that we give an appropriate error message when someone tries to use python2

Test plan:

CI.

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

facebook-github-bot commented Apr 21, 2021

💊 CI failures summary and remediations

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



1 failure not recognized by patterns:

Job Step Action
CircleCI pytorch_python_doc_build Checkout code 🔁 rerun

🚧 1 ongoing upstream failure:

These were probably caused by upstream breakages that are not fixed yet.

🚧 1 fixed upstream failure:

These were probably caused by upstream breakages that were already fixed.

Please rebase on the viable/strict branch (expand for instructions)

If your commit is older than viable/strict, run these commands:

git fetch https://github.com/pytorch/pytorch viable/strict
git rebase FETCH_HEAD

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
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
Contributor

@samestep merged this pull request in 7660cb8.

krshrimali pushed a commit to krshrimali/pytorch that referenced this pull request May 19, 2021
Summary:
This should clarify its purpose, which is:

> to make sure that we give an appropriate error message when someone tries to use python2

Pull Request resolved: pytorch#56593

Test Plan: CI.

Reviewed By: gchanan

Differential Revision: D27913086

Pulled By: samestep

fbshipit-source-id: e7555d5cab5696b19a17824383c92f25f91da2cf
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