Skip to content

Conversation

@malfet
Copy link
Contributor

@malfet malfet commented Mar 30, 2022

Summary: This would eliminate the need for build system changes when new .pyi.in files is added

Test Plan: CI

Reviewed By: seemethere

Differential Revision: D35255502

@facebook-github-bot
Copy link
Contributor

facebook-github-bot commented Mar 30, 2022

🔗 Helpful links

💊 CI failures summary and remediations

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


💚 💚 Looks good so far! There are no failures yet. 💚 💚


This comment was automatically generated by Dr. CI (expand for details).

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

Click here to manually regenerate this comment.

@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D35255502

1 similar comment
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D35255502

@malfet malfet force-pushed the export-D35255502 branch from ee6e623 to b73972e Compare March 30, 2022 18:14
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D35255502

@malfet malfet force-pushed the export-D35255502 branch from b73972e to 7f4d890 Compare March 30, 2022 21:30
)

Summary:
Pull Request resolved: pytorch#74962

This would eliminate the need for build system changes when new .pyi.in files is added

Test Plan: CI

Reviewed By: seemethere

Differential Revision: D35255502

fbshipit-source-id: e73341c4e17df8825e50f20a3c7d7a0252e757f9
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D35255502

@malfet malfet force-pushed the export-D35255502 branch from 7f4d890 to b80bc11 Compare March 31, 2022 11:07
facebook-github-bot pushed a commit that referenced this pull request Mar 31, 2022
Summary:
Pull Request resolved: #74962

This would eliminate the need for build system changes when new .pyi.in files is added

Test Plan: CI

Reviewed By: seemethere

Differential Revision: D35255502

fbshipit-source-id: 9aa84d2fc0f7933293cac4c6588da72a9d649df0
@malfet malfet deleted the export-D35255502 branch March 31, 2022 17:13
@albanD
Copy link
Collaborator

albanD commented Mar 31, 2022

Should the OSS bazel build also rely on this file?

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