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

Split GUI requirements out into their own file. #560

Merged
merged 2 commits into from Dec 4, 2019

Conversation

bcipriano
Copy link
Collaborator

Link the Issue(s) this Pull Request is related to.
Fixes #360

Summarize your change.
Split out the new file and start using it everywhere that runs GUI code.

Notably this lets us greatly simplify the RQD dockerfile.

@bcipriano bcipriano marked this pull request as ready for review December 4, 2019 00:58
@bcipriano
Copy link
Collaborator Author

AcademySoftwareFoundation/opencue.io#128 with the docs changes to match.

Copy link
Collaborator

@gregdenton gregdenton left a comment

Choose a reason for hiding this comment

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

LGTM.

@bcipriano
Copy link
Collaborator Author

@gregdenton Merging is still blocked here -- I think you just posted a comment rather than leaving a "review" -- but I'm taking your message as a review and will force the merge :)

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

Successfully merging this pull request may close these issues.

Package LICENSE file and fine-tune requirements.txt per-module
2 participants