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: link to replit run #487

Closed
wants to merge 2 commits into from
Closed

add: link to replit run #487

wants to merge 2 commits into from

Conversation

hyperupcall
Copy link

i think it would be pretty cool to let people see how this code works (and edit / preview the program) right in their browser, without any manual downloads or installation. i'm thinking we can add a 'run on repl.it' button, which redirects to something like this:

image

@ljharb
Copy link
Collaborator

ljharb commented Dec 10, 2019

We'd not want to link to something that uses an unofficial package installer, so this is a great idea if you can use npm instead of yarn (ie, if you can make the repl.it actually match this repo, which uses npm)

@hyperupcall
Copy link
Author

hyperupcall commented Dec 11, 2019

oh gotchia - yes i can match the commands

edit: how does it look

@@ -0,0 +1,2 @@
language = "nodejs"
run = "npm install && cd example && node array.js"
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
run = "npm install && cd example && node array.js"
run = "npm install && cd example && node array"

altho i wonder if we should make an index.js that requires all the other files, so we can run all the examples here?

@ljharb
Copy link
Collaborator

ljharb commented Apr 24, 2020

@eankeen did you delete your fork? :-( if so, could you try recreating it, so I can land this?

@ljharb ljharb added the needs "allow edits" PR needs "allow edits" checked label Apr 24, 2020
@hyperupcall hyperupcall closed this Nov 4, 2020
@ljharb
Copy link
Collaborator

ljharb commented Nov 4, 2020

@eankeen I'd still like to land this, but it appears your fork has been deleted.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs "allow edits" PR needs "allow edits" checked
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants