-
Notifications
You must be signed in to change notification settings - Fork 112
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
Containerize this Project for Easier Contribution #740
Comments
Good idea, our dependencies are pretty straightforward if you want to give it a go! |
Wavesrv depends on waveshell, so you can't copy just wavesrv when creating the wavesrv-builder stage. It looks like you copied both for the waveshell-builder instead of for the wavesrv-builder. I would recommend running all the go builds as a monolithic backend-builder stage |
Hey, during my local build, I discovered some issue in build-linux.md in there but even after resolving it during running client, it gives a strange error |
Ah, it looks like our documentation in the build-linux document is out of date, we use Yarn Modern for our package manager, which requires some extra setup. Check out the info in the macOS instructions regarding Yarn: https://github.com/wavetermdev/waveterm/blob/main/BUILD.md#prereqs-and-tools Specifically, you need to run the following: corepack enable
yarn install |
Is your feature request related to a problem? Please describe.
I'm always frustrated when trying to set up the development environment for this project. The process can be time-consuming and error-prone, particularly when dealing with different dependencies and system configurations.
Describe the solution you'd like
I propose containerizing this project using Docker. By providing a pre-configured Docker container, developers can quickly and easily set up a consistent development environment. This would streamline the onboarding process for new contributors and ensure that everyone is working with the same setup, reducing the likelihood of environment-related issues.
The text was updated successfully, but these errors were encountered: