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

.github: Fix blank repository variable #5086

Merged
merged 1 commit into from
Apr 10, 2024

Conversation

seemethere
Copy link
Member

This should fix a bug where the blank repository field didn't work as expected for executing checkouts / scripts correctly.

Should correct errors where users had to fill out the repository section where they should not have had to do that

This should fix a bug where the blank repository field didn't work as
expected for executing checkouts / scripts correctly.

Should correct errors where users had to fill out the repository section
where they should not have had to do that

Signed-off-by: Eli Uriegas <[email protected]>
@seemethere seemethere requested a review from a team April 10, 2024 21:14
Copy link

vercel bot commented Apr 10, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
torchci ⬜️ Ignored (Inspect) Apr 10, 2024 9:14pm

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Apr 10, 2024
@seemethere seemethere merged commit 5d3a6b9 into main Apr 10, 2024
28 of 33 checks passed
@seemethere seemethere deleted the seemethere/fix_repository_var branch April 10, 2024 21:16
facebook-github-bot pushed a commit to pytorch/executorch that referenced this pull request Apr 11, 2024
Summary:
After pytorch/test-infra#5086, the working directory is now set correctly, so `pushd` isn't needed anymore.  More importantly, trying to change the directory ends up failing all macOS CI jobs because that subdirectory doesn't exist.

Pull Request resolved: #2980

Reviewed By: larryliu0820

Differential Revision: D55996299

Pulled By: huydhn

fbshipit-source-id: 05758603d7628cc0a01fd577a49202d45c84e6c5
pytorchbot pushed a commit to pytorch/executorch that referenced this pull request Apr 12, 2024
Summary:
After pytorch/test-infra#5086, the working directory is now set correctly, so `pushd` isn't needed anymore.  More importantly, trying to change the directory ends up failing all macOS CI jobs because that subdirectory doesn't exist.

Pull Request resolved: #2980

Reviewed By: larryliu0820

Differential Revision: D55996299

Pulled By: huydhn

fbshipit-source-id: 05758603d7628cc0a01fd577a49202d45c84e6c5
(cherry picked from commit 2fc99b0)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants