-
Notifications
You must be signed in to change notification settings - Fork 1
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
Poetry setup in readme #61
base: dev
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added the steps of building the presenter UI (this causes an error about a path being registered multiple times (I assume that's the bug you're facing too)) :)
@@ -47,7 +48,9 @@ On MacOS: | |||
|
|||
### Running | |||
|
|||
To just run the server standalone without installing, run `python ./launch.py`. | |||
To run the server standalone without installing BAPSicle, you'll need to install the dependencies. | |||
Run `poetry install` in the root directory to install them. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Run `poetry install` in the root directory to install them. | |
Run `poetry install` in the root directory to install them. | |
BAPSicle also requires a built version of the [presenter UI](https://github.com/UniversityRadioYork/WebStudio) in order to use; run `npm run presenter-make` to build it (the output will be in `presenter-ui`/). |
@@ -39,6 +39,7 @@ Starting and stopping the server, as well as UI links, are available in the Syst | |||
On all platforms: | |||
|
|||
- Python 3.8 - 3.11 Tested | |||
- [Poetry](https://python-poetry.org/) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- [Poetry](https://python-poetry.org/) | |
- [Poetry](https://python-poetry.org/) | |
- NodeJS and Yarn 1.x |
When the migration was made to poetry the setup instructions in the readme were not update to reflect that change.
I've tried my best to do that having never used poetry before lmao if u can make sure these are what you expected before merge BCS I can't fully test this I have an (I assume) unrelated bug.
kthxxx