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

Feat: windows build support, more readme instructions #33

Merged
merged 5 commits into from
Jun 22, 2022

Conversation

TheyCallMeCheng
Copy link
Contributor

I added support for windows by adding another script in package.json, this error popped up when trying to build:

image

I added some instructions in the readme so it's easier for new people to start contributing.

@asrvd
Copy link
Member

asrvd commented Jun 22, 2022

works for me in windows though, @nexxeln thoughts?

@TheyCallMeCheng
Copy link
Contributor Author

TheyCallMeCheng commented Jun 22, 2022

works for me in windows though, @nexxeln thoughts?

Are you using windows WSL w/ ubuntu maybe?
rm & chmod (which are used by npm build script) shouldn't work on windows, or maybe I'm doing something wrong

README.md Outdated
Comment on lines 21 to 25
To start your app
```bash
yarn
yarn dev
```
Copy link
Member

Choose a reason for hiding this comment

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

Please remove this section

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Resolved in 44fd30a

README.md Outdated
Comment on lines 39 to 42
To start the package locally on mac/linux
```bash
npm start
```
Copy link
Member

Choose a reason for hiding this comment

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

To start the CLI locally on mac/linux

pnpm start

README.md Outdated
Comment on lines 44 to 47
To start the package on windows:
```bash
npm start-w
```
Copy link
Member

Choose a reason for hiding this comment

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

To start the package on windows:

pnpm start-w

@nexxeln
Copy link
Member

nexxeln commented Jun 22, 2022

Thank you!

@nexxeln nexxeln merged commit 835c174 into t3-oss:main Jun 22, 2022
devvianto605 pushed a commit to devvianto605/create-devviantex-nextjs-app-deprecated that referenced this pull request Jun 9, 2024
Feat: windows build support, more readme instructions
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