-
Notifications
You must be signed in to change notification settings - Fork 59
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
Update to BUILDING.md #420
Conversation
skrewde
commented
Sep 22, 2023
•
edited
Loading
edited
- Correct typos.
- Edit lines ensuring a consistent tense and pacing.
- Tweak spacing and script notes.
* Corrected typos. * Edited lines to ensure a consistent tense and pacing. * Tweaked spacing and script notes.
|
||
See the section about [Building for Production](https://vitejs.dev/guide/build.html) for more information. | ||
### `yarn build` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it would be best to just change it to something like
This command creates an optimized production build and places it in the `build` folder.
See this section about [Building for Production](https://vitejs.dev/guide/build.html) for more information.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I agree. I was unsure of the OP's intent so I kept the lines.
Edit: Also edited my commit message to reflect present tense. Read a post on best (commit) practices and I'm trying to adhere to them.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
last thing, command optimizes the production build
makes it sound like it's optimizing an existing build, but it creates the optimized production build
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missed the notification for this. Please let me know if the change I just made is sufficient.
Also, is there a better way to contact you? Discord maybe?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you can join the server https://discord.gg/DDZdqZWaHA