Skip to content

Add a CI Job to Check BC Changes in Function Schemas#26329

Closed
houseroad wants to merge 4 commits intopytorch:masterfrom
houseroad:add_bc_ci_s2
Closed

Add a CI Job to Check BC Changes in Function Schemas#26329
houseroad wants to merge 4 commits intopytorch:masterfrom
houseroad:add_bc_ci_s2

Conversation

@houseroad
Copy link
Copy Markdown
Member

@houseroad houseroad commented Sep 17, 2019

Ready for review, and results, please check https://circleci.com/gh/pytorch/pytorch/2827354?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link

Also, an experimental PR stacked on this have caught bc-breaking changes introduced by it.
#26398

@pytorchbot pytorchbot added the module: ci Related to continuous integration label Sep 17, 2019
@pytorchbot pytorchbot added the module: infra Relates to CI infrastructure label Sep 17, 2019
@houseroad houseroad force-pushed the add_bc_ci_s2 branch 7 times, most recently from 272bb4b to 47f617a Compare September 18, 2019 12:21
@houseroad houseroad changed the title [WIP] Add a CI Job to Check BC Changes in Function Schemas Add a CI Job to Check BC Changes in Function Schemas Sep 18, 2019
Comment thread .jenkins/pytorch/test.sh Outdated
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: need a space here, it shows up as 'changesfor'

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good catch

Comment thread test/backward_compatibility/check_backward_compatibility.py Outdated
Copy link
Copy Markdown
Contributor

@ezyang ezyang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@houseroad houseroad force-pushed the add_bc_ci_s2 branch 2 times, most recently from 586b2f3 to 78bb5ab Compare September 19, 2019 20:05
Copy link
Copy Markdown
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@houseroad 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

@houseroad merged this pull request in 30e7665.

laurentdupin pushed a commit to laurentdupin/pytorch that referenced this pull request Apr 24, 2026
Summary:
Ready for review, and results, please check https://circleci.com/gh/pytorch/pytorch/2827354?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link

Also, an experimental PR stacked on this have caught bc-breaking changes introduced by it.
pytorch#26398
Pull Request resolved: pytorch#26329

Reviewed By: hl475

Differential Revision: D17485668

Pulled By: houseroad

fbshipit-source-id: b10682f1785a20ea04521992e0973b1380b4dd3b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Merged module: ci Related to continuous integration module: infra Relates to CI infrastructure

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants