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

Quickstart improvements #159

Merged
merged 7 commits into from
Oct 24, 2023
Merged

Quickstart improvements #159

merged 7 commits into from
Oct 24, 2023

Conversation

gvdongen
Copy link
Contributor

@gvdongen gvdongen commented Oct 3, 2023

  • Remove redundant npm run build
  • Improve explanation of service registration
  • Don't call service greeter (more generic)

@netlify
Copy link

netlify bot commented Oct 3, 2023

Deploy Preview for docsrestatedev ready!

Name Link
🔨 Latest commit f28248f
🔍 Latest deploy log https://app.netlify.com/sites/docsrestatedev/deploys/6537bc28e7ef7c0008c63d80
😎 Deploy Preview https://deploy-preview-159--docsrestatedev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Oct 3, 2023

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: f28248f
Status:⚡️  Build in progress...

View logs

Copy link
Contributor

@AhmedSoliman AhmedSoliman left a comment

Choose a reason for hiding this comment

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

🚀

Comment on lines +33 to +36
You can run the app with `ts-node-dev` while developing:
```shell
npm run app-dev
```
Copy link
Contributor

Choose a reason for hiding this comment

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

We already run the app In step 3, right? I'm not sure if this change is needed. Maybe just leave it at npm install and let the build+run be in step 3 as the existing flow. What do you think?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The idea was to clarify that you can run with ts-node-dev while developing your services, via npm run app-dev.
Once you are done with development, you build and run with npm run app

docs/quickstart.mdx Outdated Show resolved Hide resolved
docs/quickstart.mdx Show resolved Hide resolved
@AhmedSoliman AhmedSoliman mentioned this pull request Oct 4, 2023
@gvdongen gvdongen merged commit 65aedc4 into main Oct 24, 2023
4 of 6 checks passed
@gvdongen gvdongen deleted the quickstart_improvements branch October 24, 2023 12:44
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.

2 participants