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

chore(env): added setup.sh and readme updates for working build and run #79

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

mickster04
Copy link
Contributor

I had some issues getting the build working using readme,
@Saschl helped identify the extra steps needed, This change updates the readme to point to a sh file that can be run to do all the setup steps. this in itself is simple enough and can be run command by command manually as needed.
I also added further instructions on how to install and use an older Node version.

@Lucky38i
Copy link
Member

Lucky38i commented Sep 3, 2023

Is this stale? @mickster04

@mickster04
Copy link
Contributor Author

I dunno what else I could do here?

@Lucky38i
Copy link
Member

Lucky38i commented Sep 5, 2023

I dunno what else I could do here?

Well there's a pending change. Once you amend that I can approve it and get it merged.

@Lucky38i Lucky38i changed the title added setup.sh and readme updates for working build and run chore(): added setup.sh and readme updates for working build and run Sep 5, 2023
@Lucky38i Lucky38i changed the title chore(): added setup.sh and readme updates for working build and run chore(env): added setup.sh and readme updates for working build and run Sep 5, 2023
@mickster04
Copy link
Contributor Author

Is that resolved now? I don't know what pending change you're referring to otherwise sorry.

@Lucky38i
Copy link
Member

Is that resolved now? I don't know what pending change you're referring to otherwise sorry.

Move the setup.sh into the scripts folder please.

@mickster04
Copy link
Contributor Author

Good catch, also I didn't do a branch, I hope it's ok for this change.

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.

3 participants