Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix style check CI job #3564

Closed
wants to merge 1 commit into from
Closed

Fix style check CI job #3564

wants to merge 1 commit into from

Conversation

mthrok
Copy link
Collaborator

@mthrok mthrok commented Aug 16, 2023

It seems that the default Python version was updated to 3.11.
libcst does not have binary release for 3.11, so the CI attempts to
build from source but it fails because building libcst requires Rust
compiler.

This commit fix the Python version of style check job to 3.10 so that
the issue with Rust compiler is avoided.

@pytorch-bot
Copy link

pytorch-bot bot commented Aug 16, 2023

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/audio/3564

Note: Links to docs will display an error until the docs builds have been completed.

❌ 4 New Failures

As of commit 0286f55 with merge base c593961 (image):

NEW FAILURES - The following jobs have failed:

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@mthrok mthrok force-pushed the fix-stylecheck-ci branch 3 times, most recently from ede293f to e6cfe45 Compare August 20, 2023 01:22
It seems that the default Python version was updated to 3.11.
libcst does not have binary release for 3.11, so the CI attempts to
build from source but it fails because building libcst requires Rust
compiler.

This commit fix the Python version of style check job to 3.10 so that
the issue with Rust compiler is avoided.
@mthrok mthrok force-pushed the fix-stylecheck-ci branch from e6cfe45 to 0286f55 Compare August 20, 2023 01:34
@mthrok mthrok changed the title Add rust compiler to lint job Fix style check CI job Aug 20, 2023
@mthrok mthrok marked this pull request as ready for review August 20, 2023 01:35
@facebook-github-bot
Copy link
Contributor

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

@facebook-github-bot
Copy link
Contributor

@mthrok merged this pull request in a5da0a2.

@mthrok mthrok deleted the fix-stylecheck-ci branch August 20, 2023 18:14
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.

2 participants