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

Add support for developing on Replit #37

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

cbrewster
Copy link

Hey! I wanted to see if we could get this repo working on Replit.

I added a couple config files to properly setup LSP and provide go and just in the environment and I wired up the Run button to run the full test suite. You can try this out by forking my repl here: https://replit.com/@ConnorBrewster/web5-go or, if this is merged, you can import this repo directly into a new repl and start coding right away.

@@ -0,0 +1,14 @@
run = ["just", "test"]

modules = ["go-1.21:v2-20231201-3b22c78"]
Copy link
Contributor

Choose a reason for hiding this comment

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

This repo requires Go 1.22

Copy link
Author

Choose a reason for hiding this comment

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

Let me see if we can get an updated Go module out for 1.22

Copy link
Contributor

Choose a reason for hiding this comment

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

I see you're adding Hermit to Nix, presumably to enable its use through Replit? That would definitely be our ideal path going forward as we're managing all of our toolchain versions through Hermit. But this works for now.

Copy link
Author

Choose a reason for hiding this comment

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

Yeah that's the ideal plan here, hopefully can get it into nixpkgs and available on Replit here soon

Copy link
Contributor

Choose a reason for hiding this comment

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

Thanks for doing that!

@alecthomas
Copy link
Contributor

Replit is cool BTW, I've played around with it a couple of times.

@michaelneale
Copy link

does this have the right version of go and can merge?

@cbrewster
Copy link
Author

does this have the right version of go and can merge?

Not yet, we don't have Go 1.22 available on Replit yet but we should next week

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.

None yet

3 participants