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

Unbreak sapp CI build #225

Closed
wants to merge 1 commit into from
Closed

Unbreak sapp CI build #225

wants to merge 1 commit into from

Conversation

@SamChou19815
Copy link
Contributor

@SamChou19815 SamChou19815 commented Dec 21, 2019

requirements.txt is removed in this commit
The requirements are already automatically installed during setup. Therefore, we don't need to run pip install -r requirements.txt, and we can simply remove the line in CI config.

@SamChou19815 SamChou19815 marked this pull request as ready for review Dec 21, 2019
@gbleaney
Copy link
Contributor

@gbleaney gbleaney commented Jan 5, 2020

Good catch. Thanks for the PR!

Copy link
Contributor

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

@gbleaney is landing this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@SamChou19815 SamChou19815 deleted the SamChou19815:patch-1 branch Jan 6, 2020
@facebook-github-bot
Copy link
Contributor

@facebook-github-bot facebook-github-bot commented Jan 6, 2020

@gbleaney merged this pull request in ca18cbf.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.