Skip to content

feat: Adding support to run on Github Codespaces for local development and testing. #131

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

Closed
wants to merge 1 commit into from

Conversation

alexjyong
Copy link

Signed-off-by: Alex Yong [email protected]

Please make sure to check the following tasks before opening and submitting a PR

  • I understand and have followed the contribution guide
  • I have tested my changes locally and they are working as intended

@DeclanChidlow
Copy link
Member

While this all seems to be working quite well, I do question the utility of such a thing. Given this repository runs everything with Docker containers, it isn't really suited for development.

I suppose it is handy for testing, but users wishing to test Revolt would probably just be best off checking out the already hosted instance, and the Codespaces environment isn't a great reproduction of running Revolt on a server.

Perhaps there is a use case I'm missing?

@alexjyong
Copy link
Author

In my case, I wanted to be able to run something myself quickly and easily without having to set up an account on a hosting platform (and at the time, digitalocean, which I do have an account with, was having an incident).

This can also be useful if you modify the revolt code, rebuild the docker images, and want to test it out quickly in a ephemeral environment.

I myself do a lot of work in GitHub codespaces and having something available that I can use to quickly fire up a dev environment is pretty handy.

@DeclanChidlow DeclanChidlow requested a review from insertish March 6, 2025 10:26
@IAmTomahawkx
Copy link
Member

This may be more suited towards adding onto the backend repo. This repo is specifically intended for deploying static release builds, not really for development. I'll leave it open to @insertish for final say though.

@insertish
Copy link
Member

insertish commented Apr 25, 2025

This seems more appropriate to include in a repository such as revoltchat/backend and/or revoltchat/frontend? (as Tom said)
I don't have anything else useful to add.

@alexjyong
Copy link
Author

@insertish Right on, I'll move the PR to the other repos. Thanks!

@alexjyong alexjyong closed this Apr 25, 2025
@github-project-automation github-project-automation bot moved this from 🆕 Untriaged to Closed in Pull Request Overview Apr 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

4 participants